Merge pull request #572 from SuhwanJang/fix

jhpark816 · web-flow · commit 6c3fb54e1cd9 · 2021-02-23T14:24:20.000+09:00
FIX: main_zk is not set yet in arcus_zk_client_init()
diff --git a/arcus_zk.c b/arcus_zk.c
@@ -396,7 +396,7 @@ arcus_zk_client_init(zk_info_t *zinfo)
     // ZK client ping period is recv_timeout / 3.
     arcus_conf.logger->log(EXTENSION_LOG_INFO, NULL,
             "ZooKeeper client initialized. (ZK session timeout=%d sec)\n",
-            zoo_recv_timeout(main_zk->zh)/1000);
+            zoo_recv_timeout(zinfo->zh)/1000);
     return 0;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -396,7 +396,7 @@ arcus_zk_client_init(zk_info_t *zinfo)`
`396`	`396`	`// ZK client ping period is recv_timeout / 3.`
`397`	`397`	`arcus_conf.logger->log(EXTENSION_LOG_INFO, NULL,`
`398`	`398`	`"ZooKeeper client initialized. (ZK session timeout=%d sec)\n",`
`399`		`- zoo_recv_timeout(main_zk->zh)/1000);`
	`399`	`+ zoo_recv_timeout(zinfo->zh)/1000);`
`400`	`400`	`return 0;`
`401`	`401`	`}`
`402`	`402`