-
Notifications
You must be signed in to change notification settings - Fork 672
Conversation
travis build for a fork
more travis tweaks adding link for pre-build process goveralls is out for now, also test fixes for older jar strategy bump them timeouts adding coveralls but ignore if it fails Adding back the coveralls env var
…d 3.5 zk test output to nil like before
…t in test scripts
ZK 3.5 part1 - Adding test harness in travis and test code for support for zk 3.5 and 3.4
…uild instructions
…en connected to a node that is no longer accepting connecitons
ZK 3.5 part 2 - Adding reconfig and incremental reconfig apis
1 similar comment
There are a lot of good changes in this PR. I've been integrating improvements to the upstream: https://github.com/z-division/go-zookeeper/wiki |
Thanks, If you are looking for more improvements my personal fork master branch has some more testing and bug fixes :D |
Thanks for the change. I'm not actively maintaining this repo as I haven't used Zookeeper for a very long time, but I'm trying to go through and get some of these changes in just in case there's anyone still relying on this package. |
Supersedes #205
tested on ZK 3.5.4-beta 5 node zk clusters with simple add and remove actions beyond the integration tests included in this PR.
Fixes #204