vvysotskyi opened a new pull request #2057: DRILL-7702: Update zookeeper, curator, guava, jetty-server, libthrift, httpclient, commons-compress and httpdlog-parser URL: https://github.com/apache/drill/pull/2057 # [DRILL-7702](https://issues.apache.org/jira/browse/DRILL-7702): Update zookeeper, curator, guava, jetty-server, libthrift, httpclient, commons-compress and httpdlog-parser ## Description This pull request contains 3 commits where different libraries are updated. - The first commit contains changes for updating `jetty`, `commons-compress`, `httpclient`, `httpdlog-parser`, `libthrift` and `derby` libraries. Such libraries as `log4j-core`, `commons-httpclient` and `tomcat` (`jasper-compiler`, etc.) are banned to avoid adding them occasionally as transitive dependencies to the project. - The second commit contains changes for updating ZooKeeper and Curator versions. The latest available curator version (4.3.0) doesn't support Zookeeper 3.6.0 yet, so updated Zookeeper to 3.5.7. Updating Curator helped to avoid using the outdated `org.codehaus.jackson` libraries, but they weren't added to banned dependencies since they are transitively used by other non-default supported profiles, in particular by `hbase-server` of one of non-default profile. Fixed error after the update when Drill is shutting down from distributed mode and `TransientStoreFactory` is closed after closing `CuratorFramework`, so the client was already closed. - The latest commit contains changes for updating the shaded Guava version. In new version were removed `AbstractCheckedFuture` and `CheckedFuture` classes. But since Drill actively uses the approach of mapping exceptions for `Future`, was used advice from Guava docs to move these classes into the project. Additionally, Guava had a hard dependency on `com.google.guava:failureaccess` library which has packages that match the shaded package pattern. So this library was also added to be shaded. ## Documentation NA ## Testing Tested manually, started Drill in distributed mode, connected to Apache Zookeeper Docker container, checked custom authenticator with JDBC driver.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to 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
