-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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-3865: fix backward-compatibility for ZooKeeperServer constructor #1383
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
retest ant build |
I see the ant job fails again around the C client build, without an obvious error message. I've seen this before... Anyway, it shouldn't be related to the PR I think. |
retest maven build |
retest ant build |
@eolivelli do you think we can merge this? Only the ant job failed with some unrelated problems (and also the change is pretty harmless). I'm happy to merge if you agree but don't have the time. |
I can merge it tomorrow |
merging |
…uctor In the fix of ZOOKEEPER-3829 I accidentally changed a public constructor of ZooKeeper server. I didn’t realize Curator is using it for testing (and maybe other Apache projects are doing so as well). In this PR I restore the original constructor for branch-3.5 making it compatible with e.g. Curator 4.3. Author: Mate Szalay-Beko <symat@apache.org> Reviewers: Enrico Olivelli <eolivelli@apache.org> Closes #1383 from symat/ZOOKEEPER-3865-branch-3.5
pushed to branch-3.5 |
thanks!! |
In the fix of ZOOKEEPER-3829 I accidentally changed a public constructor of ZooKeeper server. I didn’t realize Curator is using it for testing (and maybe other Apache projects are doing so as well).
In this PR I restore the original constructor for branch-3.5 making it compatible with e.g. Curator 4.3.