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.


---

Reply via email to