KazydubB commented on a change in pull request #1337: Upgrade ZooKeeper patch
version to 3.4.12 and add Apache Curator to dependencyManagement
URL: https://github.com/apache/drill/pull/1337#discussion_r199242382
##########
File path: pom.xml
##########
@@ -1552,6 +1553,11 @@
</exclusion>
</exclusions>
</dependency>
+ <dependency>
+ <groupId>org.apache.zookeeper</groupId>
+ <artifactId>zookeeper</artifactId>
Review comment:
I've tried upgrading all curator dependencies to 2.12.0, it seems to work as
expected but curator-client's size is much larger than current one's, so one
needs to update max size of drill-jdbc-all module by extra 1-2 MB. It is better
to be handled as another task.
However I did add curator dependencies to dependencyManagement explicitly.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
