[GitHub] zookeeper pull request #377: [ZOOKEEPER-2901] TTL Nodes don't work with Serv...

2017-10-20 Thread DanBenediktson
Github user DanBenediktson commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/377#discussion_r146089767 --- Diff: conf/zoo_sample.cfg --- @@ -6,6 +6,10 @@ initLimit=10 # The number of ticks that can pass between # sending a request

[GitHub] zookeeper pull request #342: ZOOKEEPER-2488: Synchronized access to shutting...

2017-08-23 Thread DanBenediktson
Github user DanBenediktson commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/342#discussion_r134784530 --- Diff: src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java --- @@ -1155,6 +1134,19 @@ public void run

[GitHub] zookeeper pull request #330: ZOOKEEPER-2471: ZK Java client should not count...

2017-08-09 Thread DanBenediktson
GitHub user DanBenediktson opened a pull request: https://github.com/apache/zookeeper/pull/330 ZOOKEEPER-2471: ZK Java client should not count sleep time as connect time ClientCnxnSocket uses a member variable "now" to track the current time, but does not update