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>