[GitHub] zookeeper pull request #652: ZOOKEEPER-3156: Add in option to canonicalize h...

[lvfangmin]

Github user lvfangmin commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/652#discussion_r221836418
  
    --- Diff: src/java/main/org/apache/zookeeper/ClientCnxn.java ---
    @@ -793,7 +794,87 @@ public RWServerFoundException(String msg) {
                 super(msg);
             }
         }
    -    
    +
    +    static class MockableInetSocketAddress {
    --- End diff --
    
    It's adding too much complexity to the code in order to test it, are we 
able to create the stub to extend the InetSocketAddress in the test case itself?


---