[FLINK-16163][build] Migrate to flink-shaded-zookeeper #11143
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Based on #11131.
With this PR the project now uses the zookeeper 3.4 version provided by flink-shaded. This dependency also bundles curator 4, rendering flink-shaded-curator obsolete and implicitly addressing FLINK-14177.
The vast majority of changes in this PR are imports.
The only notable .java file is
ZooKeeperLeaderElectionTest
, which adjusts some mocking behavior. With curator 4 the mocking resulted in errors, but I admittedly have not looked into it too much.Beyond that, there is now a dependency management entry in the root pom for flink-shaded-zookeeper, the flink-runtime dependencies on zk/curator were replaced with the flink-shaded dependency. flink-runtime no longer bundles zookeeper; this responsibility is now deferred to flink-dist.