[GitHub] zookeeper pull request #545: ZOOKEEPER-2261 When only secureClientPort is co...

2018-09-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zookeeper/pull/545 ---

[GitHub] zookeeper pull request #545: ZOOKEEPER-2261 When only secureClientPort is co...

2018-08-08 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/545#discussion_r208507660 --- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java --- @@ -122,8 +122,8 @@ final Map outstandingChangesForPath =

[GitHub] zookeeper pull request #545: ZOOKEEPER-2261 When only secureClientPort is co...

2018-08-07 Thread hanm
Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/545#discussion_r208451942 --- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java --- @@ -122,8 +122,8 @@ final Map outstandingChangesForPath = n

[GitHub] zookeeper pull request #545: ZOOKEEPER-2261 When only secureClientPort is co...

2018-08-07 Thread hanm
Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/545#discussion_r208451880 --- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java --- @@ -122,8 +122,8 @@ final Map outstandingChangesForPath = n

[GitHub] zookeeper pull request #545: ZOOKEEPER-2261 When only secureClientPort is co...

2018-07-30 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/545#discussion_r206120103 --- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java --- @@ -866,6 +866,9 @@ public void setServerCnxnFactory(ServerCnxnFactory facto

[GitHub] zookeeper pull request #545: ZOOKEEPER-2261 When only secureClientPort is co...

2018-07-29 Thread hanm
Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/545#discussion_r206012592 --- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java --- @@ -866,6 +866,9 @@ public void setServerCnxnFactory(ServerCnxnFactory factory)

[GitHub] zookeeper pull request #545: ZOOKEEPER-2261 When only secureClientPort is co...

2018-07-27 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/545#discussion_r205729889 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/UtilTest.java --- @@ -0,0 +1,91 @@ +/** --- End diff -- I had to make a

[GitHub] zookeeper pull request #545: ZOOKEEPER-2261 When only secureClientPort is co...

2018-07-27 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/545#discussion_r205729651 --- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java --- @@ -866,6 +866,9 @@ public void setServerCnxnFactory(ServerCnxnFactory facto

[GitHub] zookeeper pull request #545: ZOOKEEPER-2261 When only secureClientPort is co...

2018-07-26 Thread breed
Github user breed commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/545#discussion_r205663866 --- Diff: src/java/test/org/apache/zookeeper/server/persistence/UtilTest.java --- @@ -0,0 +1,91 @@ +/** --- End diff -- did you mean for th

[GitHub] zookeeper pull request #545: ZOOKEEPER-2261 When only secureClientPort is co...

2018-07-26 Thread breed
Github user breed commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/545#discussion_r205663822 --- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java --- @@ -866,6 +866,9 @@ public void setServerCnxnFactory(ServerCnxnFactory factory)

[GitHub] zookeeper pull request #545: ZOOKEEPER-2261 When only secureClientPort is co...

2018-07-26 Thread hanm
Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/545#discussion_r205662651 --- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java --- @@ -866,6 +866,9 @@ public void setServerCnxnFactory(ServerCnxnFactory factory)

[GitHub] zookeeper pull request #545: ZOOKEEPER-2261 When only secureClientPort is co...

2018-06-15 Thread anmolnar
GitHub user anmolnar opened a pull request: https://github.com/apache/zookeeper/pull/545 ZOOKEEPER-2261 When only secureClientPort is configured connections, configuration, connection_stat_reset, and stats admin commands throw NullPointerException Root cause of the issue is that pr