[
https://issues.apache.org/jira/browse/TRAFODION-2588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15966991#comment-15966991
]
ASF GitHub Bot commented on TRAFODION-2588:
-------------------------------------------
Github user venkat1m commented on a diff in the pull request:
https://github.com/apache/incubator-trafodion/pull/1061#discussion_r111296731
--- Diff: core/sqf/monitor/linux/zclient.cxx ---
@@ -53,6 +53,9 @@
// The monitors register their znodes under the cluster znode
#define ZCLIENT_CLUSTER_ZNODE "/cluster"
+// zookeeper connection retry count
+#define ZOOKEEPER_RETRY_COUNT 3
--- End diff --
Wouldn't it be better to make this an environment variable or monitor
config setting that can be changed instead of a hardcoded retry count of 3.
> monitor failed to start if part of zookeeper server is down
> -----------------------------------------------------------
>
> Key: TRAFODION-2588
> URL: https://issues.apache.org/jira/browse/TRAFODION-2588
> Project: Apache Trafodion
> Issue Type: Bug
> Components: foundation
> Reporter: Eason Zhang
> Assignee: Eason Zhang
>
> monitor gets zookpper node list from env $ZOOKEEPER_NODES. If one of the
> zookeeper server in this node list is down, monitor will not start with
> 'ZCONNECTIONLOSS' error:
> 2017-04-11 17:15:49,351, ERROR, ZOO, Node Number: 4294967295,, PIN: 21106 ,
> Process Name: zooclient,,, TID: 21106, Message ID: 101370401,
> [CZClient::MakeClusterZNodes], zoo_exists() failed with error ZCONNECTIONLOSS
> That is because zookeeper c client randomly picks up a zk server to connect
> with based on its own node selection algorithm. So the simple solution for
> monitor is that we do a retry when connection failed, let zookeeper c client
> to pick up another server from the list.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
