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

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


The following commit(s) were added to refs/heads/main by this push:
     new 902888893 ORC-1955: Make `commons-lang3` as a test dependency 
explicitly
902888893 is described below

commit 902888893b024927e3274cbda9391619f890aebb
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Mon Jul 14 10:14:15 2025 -0700

    ORC-1955: Make `commons-lang3` as a test dependency explicitly
    
    ### What changes were proposed in this pull request?
    
    This PR aims to make `commons-lang3` test dependency
    
    ### Why are the changes needed?
    
    Since ORC-1791, we remove `commons-lang3` dependency requirement. This is 
only used during testing or `bench` module.
    - #2053
    
    ### How was this patch tested?
    
    Pass the CIs.
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No.
    
    Closes #2327 from dongjoon-hyun/ORC-1955.
    
    Authored-by: Dongjoon Hyun <[email protected]>
    Signed-off-by: Dongjoon Hyun <[email protected]>
---
 java/bench/spark/pom.xml | 2 ++
 java/pom.xml             | 1 +
 2 files changed, 3 insertions(+)

diff --git a/java/bench/spark/pom.xml b/java/bench/spark/pom.xml
index b0c3e3d11..0ad26152a 100644
--- a/java/bench/spark/pom.xml
+++ b/java/bench/spark/pom.xml
@@ -51,6 +51,8 @@
     <dependency>
       <groupId>org.apache.commons</groupId>
       <artifactId>commons-lang3</artifactId>
+      <!-- Spark uses org.apache.commons.lang3.SystemUtils -->
+      <scope>compile</scope>
     </dependency>
     <dependency>
       <groupId>org.apache.hive</groupId>
diff --git a/java/pom.xml b/java/pom.xml
index 706958ff8..3767081ad 100644
--- a/java/pom.xml
+++ b/java/pom.xml
@@ -152,6 +152,7 @@
         <groupId>org.apache.commons</groupId>
         <artifactId>commons-lang3</artifactId>
         <version>3.18.0</version>
+        <scope>test</scope>
       </dependency>
       <dependency>
         <groupId>io.airlift</groupId>

Reply via email to