[GitHub] [incubator-livy] yiheng opened a new pull request #239: [Minor][Thrift] Exclude curator in thrift server pom to avoid conflict jars

[GitBox <gi...@apache.org>]

yiheng opened a new pull request #239: [Minor][Thrift] Exclude curator in thrift server pom to avoid conflict jars
URL: https://github.com/apache/incubator-livy/pull/239
 
 
   ## What changes were proposed in this pull request?
   Currently, thrift server has a dependency of curator-client:2.12.0 through the hive service. After the build, a `curator-client-2.12.0.jar` file will be generated in the `jars` folder. It is conflicted with the `curator-client-2.7.1.jar` file, which is used by livy server.
   
   We observed that in some JDK, the `curator-client-2.12.0.jar` is loaded before the `curator-client-2.7.1.jar`, and will crash the recovery enabled livy server.
   
   In this patch, we exclude the `org.apache.curator` modules from the hive dependency. 
   
   ## How was this patch tested?
   Manual test and existing UT/ITs
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services