Skip to content

Commit

Permalink
[LIVY-690][THRIFT] Exclude curator in thrift server pom to avoid conf…
Browse files Browse the repository at this point in the history
…lict jars

## 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

Author: Yiheng Wang <[email protected]>

Closes apache#239 from yiheng/exclude_curator.
  • Loading branch information
yiheng authored and jerryshao committed Sep 29, 2019
1 parent 0ea4a14 commit 0804c8e
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions thriftserver/server/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,10 @@
<groupId>org.eclipse.jetty</groupId>
<artifactId>*</artifactId>
</exclusion>
<exclusion>
<groupId>org.apache.curator</groupId>
<artifactId>*</artifactId>
</exclusion>
</exclusions>
</dependency>

Expand Down

0 comments on commit 0804c8e

Please sign in to comment.