[GitHub] zookeeper pull request #119: ZOOKEEPER-2251:Add Client side packet response ...

[hanm]

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

    https://github.com/apache/zookeeper/pull/119#discussion_r204626490
  
    --- Diff: src/java/main/org/apache/zookeeper/KeeperException.java ---
    @@ -843,4 +847,13 @@ public ReconfigDisabledException(String path) {
                 super(Code.RECONFIGDISABLED, path);
             }
         }
    +
    +    /**
    +     * @see Code#TIMEOUT
    +     */
    +    public static class TimeoutException extends KeeperException {
    --- End diff --
    
    Similarly, use a more descriptive name like 
`ServerResponseTimeoutException` sounds better to me.


---