This is an automated email from the ASF dual-hosted git repository.

jshao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-livy.git


The following commit(s) were added to refs/heads/master by this push:
     new 1f64264  [LIVY-708][SERVER] Align curator jars version
1f64264 is described below

commit 1f64264135409f4a9c7f094660c7d4d6e4130a34
Author: yihengwang <yihengw...@tencent.com>
AuthorDate: Tue Nov 12 09:53:44 2019 +0800

    [LIVY-708][SERVER] Align curator jars version
    
    ## What changes were proposed in this pull request?
    Livy server has dependency of Apache Curator through hadoop client. 
However, the versions of the curator jars are not aligned. Here're the curator 
jars after build
    
    * curator-client-2.7.1.jar
    * curator-framework-2.7.1.jar
    * curator-recipes-2.6.0.jar
    This will cause Method not found issue in some case:
    ```
    Exception in thread "main" java.lang.NoSuchMethodError:
    org.apache.curator.utils.PathUtils.validatePath(Ljava/lang/String;)V
    ```
    This patch specify the version of curator-recipes to 2.7.1.
    
    ## How was this patch tested?
    Manually test in the env where no such method exception are thrown.
    Existing UT/IT
    
    Author: yihengwang <yihengw...@tencent.com>
    
    Closes #256 from yiheng/fix_708.
---
 pom.xml        | 1 +
 server/pom.xml | 6 ++++++
 2 files changed, 7 insertions(+)

diff --git a/pom.xml b/pom.xml
index 5bcf1a3..2f6dd09 100644
--- a/pom.xml
+++ b/pom.xml
@@ -124,6 +124,7 @@
     <!-- Required for testing LDAP integration -->
     <apacheds.version>2.0.0-M21</apacheds.version>
     <ldap-api.version>1.0.0-M33</ldap-api.version>
+    <curator.version>2.7.1</curator.version>
 
     <!--
       Properties for the copyright header style checks. Modules that use the 
ASF header
diff --git a/server/pom.xml b/server/pom.xml
index 8e797f2..25c4918 100644
--- a/server/pom.xml
+++ b/server/pom.xml
@@ -118,6 +118,12 @@
     </dependency>
 
     <dependency>
+      <groupId>org.apache.curator</groupId>
+      <artifactId>curator-recipes</artifactId>
+      <version>${curator.version}</version>
+    </dependency>
+
+    <dependency>
       <groupId>org.apache.httpcomponents</groupId>
       <artifactId>httpclient</artifactId>
       <scope>test</scope>

Reply via email to