Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zookeeper Driver Update #106

Open
MarianaLeal opened this issue Dec 17, 2021 · 3 comments
Open

Zookeeper Driver Update #106

MarianaLeal opened this issue Dec 17, 2021 · 3 comments

Comments

@MarianaLeal
Copy link

Hello!

Are there any thoughts of updating the Zookeeper driver version? This is currently using 3.4.5 and there are a lot of newer versions with bug fixes and improvements. I believe the minor version more up to date would be 3.4.14 (https://zookeeper.apache.org/doc/).

I was just wondering if this is something you're considering to do in a close future or if there any concerns with the driver's version update.

Thanks a lot!

@MarianaLeal
Copy link
Author

Providing more context to the request:

My team is struggling with an issue where our Zookeeper clients are unable to reconnect to the cluster when new ZK nodes join in with different IP addresses than the older nodes' IPs.
This issue has already been reported in the Zookeeper library and has been fixed in version 3.4.13. Therefore my request ☝️

@malmckay
Copy link
Member

malmckay commented Jan 4, 2022

@MarianaLeal I am investigating updating to 3.4.13 in this PR: #107

If you are able to reproduce, can you try that branch?

@MarianaLeal
Copy link
Author

Hello @malmckay , sorry for the delay.

It is hard to reproduce the issue using your test branch due to the nature of how our dependency tree is built.
I was able to run our project with the test branch with success so I'd say that, from our side, the driver update is approved 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants