Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/451#discussion_r191252989 --- Diff: src/java/test/org/apache/zookeeper/test/ClientPortBindTest.java --- @@ -102,9 +102,9 @@ public void testBindByAddress() throws Exception { startSignal = new CountDownLatch(1); ZooKeeper zk = new ZooKeeper(HOSTPORT, CONNECTION_TIMEOUT, this); try { - startSignal.await(CONNECTION_TIMEOUT, --- End diff -- Sure, I agree. I think that was a suggestion from @afine and it's completely reasonable.
---