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

zhengchenyu pushed a commit to branch branch-0.9
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git


The following commit(s) were added to refs/heads/branch-0.9 by this push:
     new 2a34b03f5 [#1699][FOLLOWUP] fix(client): Add commons-collections4 
dependencies in shaded clients (#1742)
2a34b03f5 is described below

commit 2a34b03f55153e8620c9b278711ac7f611e42734
Author: RickyMa <rick...@tencent.com>
AuthorDate: Mon Jun 3 14:50:03 2024 +0800

    [#1699][FOLLOWUP] fix(client): Add commons-collections4 dependencies in 
shaded clients (#1742)
    
    ### What changes were proposed in this pull request?
    
    Add `commons-collections4` dependencies in shaded clients.
    
    ### Why are the changes needed?
    
    After https://github.com/apache/incubator-uniffle/pull/1700, in some old 
versions of Spark, the `commons-collections4` dependency could be missing. Add 
the potential missing dependency to improve robustness.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Existing UTs. Tested in our env.
---
 client-mr/core/pom.xml             | 1 +
 client-spark/spark2-shaded/pom.xml | 1 +
 client-spark/spark3-shaded/pom.xml | 1 +
 client-tez/pom.xml                 | 1 +
 4 files changed, 4 insertions(+)

diff --git a/client-mr/core/pom.xml b/client-mr/core/pom.xml
index 305807878..bfda7ba6b 100644
--- a/client-mr/core/pom.xml
+++ b/client-mr/core/pom.xml
@@ -159,6 +159,7 @@
                                     
<include>org.roaringbitmap:RoaringBitmap</include>
                                     <include>org.roaringbitmap:shims</include>
                                     <include>net.jpountz.lz4:lz4</include>
+                                    
<include>org.apache.commons:commons-collections4</include>
                                 </includes>
                             </artifactSet>
                             
<finalName>${project.artifactId}-${project.version}</finalName>
diff --git a/client-spark/spark2-shaded/pom.xml 
b/client-spark/spark2-shaded/pom.xml
index b8b83aece..7bec7f99b 100644
--- a/client-spark/spark2-shaded/pom.xml
+++ b/client-spark/spark2-shaded/pom.xml
@@ -68,6 +68,7 @@
                   
<include>com.fasterxml.jackson.core:jackson-annotations</include>
                   <include>org.roaringbitmap:RoaringBitmap</include>
                   <include>org.roaringbitmap:shims</include>
+                  <include>org.apache.commons:commons-collections4</include>
                 </includes>
               </artifactSet>
               <finalName>${project.artifactId}-${project.version}</finalName>
diff --git a/client-spark/spark3-shaded/pom.xml 
b/client-spark/spark3-shaded/pom.xml
index 0c1717eb2..d972b9d8d 100644
--- a/client-spark/spark3-shaded/pom.xml
+++ b/client-spark/spark3-shaded/pom.xml
@@ -68,6 +68,7 @@
                   
<include>com.fasterxml.jackson.core:jackson-annotations</include>
                   <include>org.roaringbitmap:RoaringBitmap</include>
                   <include>org.roaringbitmap:shims</include>
+                  <include>org.apache.commons:commons-collections4</include>
                 </includes>
               </artifactSet>
               <finalName>${project.artifactId}-${project.version}</finalName>
diff --git a/client-tez/pom.xml b/client-tez/pom.xml
index 748c33aa6..d2a60d996 100644
--- a/client-tez/pom.xml
+++ b/client-tez/pom.xml
@@ -174,6 +174,7 @@
                                     
<include>org.roaringbitmap:RoaringBitmap</include>
                                     <include>org.roaringbitmap:shims</include>
                                     <include>net.jpountz.lz4:lz4</include>
+                                    
<include>org.apache.commons:commons-collections4</include>
                                 </includes>
                             </artifactSet>
                             
<finalName>${project.artifactId}-${project.version}</finalName>

Reply via email to