Github user phunt commented on the issue:
https://github.com/apache/zookeeper/pull/409
+1 thanks Andor. I committed this to branch-3.4 - you'll need to close this
PR manually.
---
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/409
@phunt Build is green now. Please review & commit if you have a chance.
---
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/409
@phunt Patch has been updated with the trunk review changes. Please take a
look.
---
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/409
@phunt Yes, I'm working on it.
---
Github user phunt commented on the issue:
https://github.com/apache/zookeeper/pull/409
@anmolnar does this need to be updated to reflect the changes in #420 ?
(looks that way to me...)
---
Github user phunt commented on the issue:
https://github.com/apache/zookeeper/pull/409
Same comments as in #420
---
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/409
@phunt Do you think we can merge this now into branch-3.4?
---
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/409
@phunt would you please kindly commit this patch?
---
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/409
Thanks @afine . I've updated the PR.
---