[GitHub] zookeeper pull request #544: ZOOKEEPER-3009 : fix the related bugs in branch...

2018-07-26 Thread lujiefsi
Github user lujiefsi closed the pull request at: https://github.com/apache/zookeeper/pull/544 ---

[GitHub] zookeeper pull request #544: ZOOKEEPER-3009 : fix the related bugs in branch...

2018-07-21 Thread lvfangmin
Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/544#discussion_r204217429 --- Diff: src/java/main/org/apache/zookeeper/server/NIOServerCnxnFactory.java --- @@ -129,11 +129,14 @@ public int getLocalPort(){ return s

[GitHub] zookeeper pull request #544: ZOOKEEPER-3009 : fix the related bugs in branch...

2018-06-17 Thread lujiefsi
Github user lujiefsi commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/544#discussion_r195967959 --- Diff: src/java/main/org/apache/zookeeper/server/NIOServerCnxnFactory.java --- @@ -165,9 +168,13 @@ public void removeCnxn(NIOServerCnxn cnxn) {

[GitHub] zookeeper pull request #544: ZOOKEEPER-3009 : fix the related bugs in branch...

2018-06-17 Thread hanm
Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/544#discussion_r195958750 --- Diff: src/java/main/org/apache/zookeeper/server/NIOServerCnxnFactory.java --- @@ -165,9 +168,13 @@ public void removeCnxn(NIOServerCnxn cnxn) {

[GitHub] zookeeper pull request #544: ZOOKEEPER-3009 : fix the related bugs in branch...

2018-06-15 Thread lujiefsi
GitHub user lujiefsi opened a pull request: https://github.com/apache/zookeeper/pull/544 ZOOKEEPER-3009 : fix the related bugs in branch-3.4 The same problem that described in ZOOKEEPER-3009 also exists in branch-3.4, just as @hanm said. I pull a new request to fix it. The patch ju