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 a7c405934 ORC-1965: Ban `org.apache.commons.lang` package
a7c405934 is described below

commit a7c405934f0cb322c24c9bf5524a2804fee4884b
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Tue Jul 29 23:48:32 2025 -0700

    ORC-1965: Ban `org.apache.commons.lang` package
    
    ### What changes were proposed in this pull request?
    
    This PR aims to ban `Commons Lang` in favor of `Commons Lang3`.
    
    ### Why are the changes needed?
    
    Apache ORC should use only `Apache Commons Lang3` in the test code.
    
    ### How was this patch tested?
    
    Pass the CIs with the newly added Checkstyle rule.
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No.
    
    Closes #2343 from dongjoon-hyun/ORC-1965.
    
    Authored-by: Dongjoon Hyun <[email protected]>
    Signed-off-by: Dongjoon Hyun <[email protected]>
---
 .../java/org/apache/orc/bench/hive/rowfilter/RowFilterInputState.java  | 2 +-
 java/checkstyle.xml                                                    | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git 
a/java/bench/hive/src/java/org/apache/orc/bench/hive/rowfilter/RowFilterInputState.java
 
b/java/bench/hive/src/java/org/apache/orc/bench/hive/rowfilter/RowFilterInputState.java
index dede8f95d..e63e36926 100644
--- 
a/java/bench/hive/src/java/org/apache/orc/bench/hive/rowfilter/RowFilterInputState.java
+++ 
b/java/bench/hive/src/java/org/apache/orc/bench/hive/rowfilter/RowFilterInputState.java
@@ -17,7 +17,7 @@
  */
 package org.apache.orc.bench.hive.rowfilter;
 
-import org.apache.commons.lang.reflect.FieldUtils;
+import org.apache.commons.lang3.reflect.FieldUtils;
 import org.apache.hadoop.conf.Configuration;
 import org.apache.hadoop.fs.FileSystem;
 import org.apache.hadoop.fs.Path;
diff --git a/java/checkstyle.xml b/java/checkstyle.xml
index 4299bd59f..26a3918be 100644
--- a/java/checkstyle.xml
+++ b/java/checkstyle.xml
@@ -61,5 +61,8 @@
       <property name="lineWrappingIndentation" value="4"/>
       <property name="arrayInitIndent" value="2"/>
     </module>
+    <module name="IllegalImport">
+      <property name="illegalPkgs" value="org.apache.commons.lang" />
+    </module>
   </module>
 </module>

Reply via email to