Github user lvfangmin commented on the issue:
https://github.com/apache/zookeeper/pull/682
Thanks @anmolnar, I've merged this onto master.
---
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/682
If there's no objection, I'll get this in tomorrow.
---
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/682
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2520/
---
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/682
retest this please
---
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/682
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2519/
---
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/682
@lvfangmin Moved bunch of common code to Before/After.
Also changed the logic to call `setData` in order to verify
`NodeDataChanged` event.
---
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/682
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2509/
---