[GitHub] zookeeper issue #549: ZOOKEEPER-2873 abort startup on invalid ports

2018-07-04 Thread anmolnar 
Github user anmolnar commented on the issue:

https://github.com/apache/zookeeper/pull/549
  
Committed to master and 3.5 branches.
Thanks @nkalmar !


---

[GitHub] zookeeper issue #549: ZOOKEEPER-2873 abort startup on invalid ports

2018-06-27 Thread eolivelli 
Github user eolivelli commented on the issue:

https://github.com/apache/zookeeper/pull/549
  
Doesn't the merge script do a squash?


---

[GitHub] zookeeper issue #549: ZOOKEEPER-2873 abort startup on invalid ports

2018-06-26 Thread nkalmar 
Github user nkalmar commented on the issue:

https://github.com/apache/zookeeper/pull/549
  
Itt introduces test failures in 
org.apache.zookeeper.test.ReconfigTest.testUnspecifiedClientAddress, I'm 
working on it!


---