You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@celeborn.apache.org by ch...@apache.org on 2024/04/24 03:53:05 UTC

(celeborn) branch main updated: [CELEBORN-1380] leveldbjni uses org.openlabtesting.leveldbjni to support linux aarch64 platform for leveldb

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

chengpan pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/celeborn.git


The following commit(s) were added to refs/heads/main by this push:
     new 9110eab99 [CELEBORN-1380] leveldbjni uses org.openlabtesting.leveldbjni to support linux aarch64 platform for leveldb
9110eab99 is described below

commit 9110eab9968be0f07c73e6e90d90a1b5b6bcf00e
Author: SteNicholas <pr...@163.com>
AuthorDate: Wed Apr 24 11:52:56 2024 +0800

    [CELEBORN-1380] leveldbjni uses org.openlabtesting.leveldbjni to support linux aarch64 platform for leveldb
    
    ### What changes were proposed in this pull request?
    
    Dependency leveldbjni uses `org.openlabtesting.leveldbjni` to support linux aarch64 platform for leveldb.
    
    ### Why are the changes needed?
    
    Celeborn worker could not start on arm arch devices if db backend is `LevelDB`, which should support leveldbjni on the aarch64 platform.
    
    aarch64 uses `org.openlabtesting.leveldbjni:leveldbjni-all.1.8`, and other platforms use `org.fusesource.leveldbjni:leveldbjni-all.1.8`. Meanwhile, because some hadoop dependencies packages are also depend on `org.fusesource.leveldbjni:leveldbjni-all`, but hadoop merge the similar change on trunk, details see
    [HADOOP-16614](https://issues.apache.org/jira/browse/HADOOP-16614), therefore it should exclude the dependency of `org.fusesource.leveldbjni` for these hadoop packages related.
    
    Backport:
    
    - https://github.com/apache/spark/pull/26636
    - https://github.com/apache/spark/pull/31036
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    No.
    
    Closes #2476 from SteNicholas/CELEBORN-1380.
    
    Authored-by: SteNicholas <pr...@163.com>
    Signed-off-by: Cheng Pan <ch...@apache.org>
---
 common/pom.xml              | 2 +-
 pom.xml                     | 6 +++++-
 project/CelebornBuild.scala | 2 +-
 worker/pom.xml              | 2 +-
 4 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/common/pom.xml b/common/pom.xml
index b7ae4cfc1..ab9dba6b1 100644
--- a/common/pom.xml
+++ b/common/pom.xml
@@ -87,7 +87,7 @@
       <artifactId>netty-all</artifactId>
     </dependency>
     <dependency>
-      <groupId>org.fusesource.leveldbjni</groupId>
+      <groupId>org.openlabtesting.leveldbjni</groupId>
       <artifactId>leveldbjni-all</artifactId>
     </dependency>
     <dependency>
diff --git a/pom.xml b/pom.xml
index 5e71136ee..357bb1839 100644
--- a/pom.xml
+++ b/pom.xml
@@ -394,7 +394,7 @@
         </exclusions>
       </dependency>
       <dependency>
-        <groupId>org.fusesource.leveldbjni</groupId>
+        <groupId>org.openlabtesting.leveldbjni</groupId>
         <artifactId>leveldbjni-all</artifactId>
         <version>${leveldb.version}</version>
       </dependency>
@@ -1198,6 +1198,10 @@
               <groupId>org.apache.hadoop</groupId>
               <artifactId>hadoop-yarn-server-common</artifactId>
             </exclusion>
+            <exclusion>
+              <groupId>org.fusesource.leveldbjni</groupId>
+              <artifactId>leveldbjni-all</artifactId>
+            </exclusion>
           </exclusions>
         </dependency>
       </dependencies>
diff --git a/project/CelebornBuild.scala b/project/CelebornBuild.scala
index 55509f00d..c5a928b0a 100644
--- a/project/CelebornBuild.scala
+++ b/project/CelebornBuild.scala
@@ -111,7 +111,7 @@ object Dependencies {
   val ioDropwizardMetricsJvm = "io.dropwizard.metrics" % "metrics-jvm" % metricsVersion
   val ioNetty = "io.netty" % "netty-all" % nettyVersion excludeAll(
     ExclusionRule("io.netty", "netty-handler-ssl-ocsp"))
-  val leveldbJniAll = "org.fusesource.leveldbjni" % "leveldbjni-all" % leveldbJniVersion
+  val leveldbJniAll = "org.openlabtesting.leveldbjni" % "leveldbjni-all" % leveldbJniVersion
   val log4j12Api = "org.apache.logging.log4j" % "log4j-1.2-api" % log4j2Version
   val log4jSlf4jImpl = "org.apache.logging.log4j" % "log4j-slf4j-impl" % log4j2Version
   val lz4Java = "org.lz4" % "lz4-java" % lz4JavaVersion
diff --git a/worker/pom.xml b/worker/pom.xml
index 3821fe9da..b255569da 100644
--- a/worker/pom.xml
+++ b/worker/pom.xml
@@ -49,7 +49,7 @@
       <version>${project.version}</version>
     </dependency>
     <dependency>
-      <groupId>org.fusesource.leveldbjni</groupId>
+      <groupId>org.openlabtesting.leveldbjni</groupId>
       <artifactId>leveldbjni-all</artifactId>
     </dependency>
     <dependency>