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 76205585b ORC-1927: Add Java `25-ea` test coverage for `tools` module
76205585b is described below
commit 76205585beb4e50dcab965924b719e34eb6678a1
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Tue Jun 17 18:25:09 2025 -0700
ORC-1927: Add Java `25-ea` test coverage for `tools` module
### What changes were proposed in this pull request?
This PR aims to add `Java 25-ea` test coverage for `tools` module in
addition to the existing `shims` and `core` modules.
### Why are the changes needed?
To improve a test coverage.
### How was this patch tested?
Pass the CIs.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #2282 from dongjoon-hyun/ORC-1927.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
.github/workflows/build_and_test.yml | 4 ++--
java/tools/src/java/org/apache/orc/tools/ColumnSizes.java | 4 +++-
java/tools/src/java/org/apache/orc/tools/Driver.java | 1 +
java/tools/src/java/org/apache/orc/tools/FileDump.java | 1 +
java/tools/src/java/org/apache/orc/tools/RowCount.java | 4 +++-
5 files changed, 10 insertions(+), 4 deletions(-)
diff --git a/.github/workflows/build_and_test.yml
b/.github/workflows/build_and_test.yml
index 36e7020be..56bd9f21c 100644
--- a/.github/workflows/build_and_test.yml
+++ b/.github/workflows/build_and_test.yml
@@ -101,8 +101,8 @@ jobs:
cd java
# JDK 25 Build
./mvnw package -DskipTests
- # JDK 25 Test: shims and core modules
- ./mvnw package --pl core --am
+ # JDK 25 Test: shims, core, tools modules
+ ./mvnw package --pl tools --am
else
mkdir build
cd build
diff --git a/java/tools/src/java/org/apache/orc/tools/ColumnSizes.java
b/java/tools/src/java/org/apache/orc/tools/ColumnSizes.java
index 2347ac744..be9066583 100644
--- a/java/tools/src/java/org/apache/orc/tools/ColumnSizes.java
+++ b/java/tools/src/java/org/apache/orc/tools/ColumnSizes.java
@@ -216,7 +216,9 @@ public class ColumnSizes {
}
public static void main(String[] args) throws Exception {
- main(new Configuration(), args);
+ Configuration conf = new Configuration();
+ conf.set("fs.file.impl.disable.cache", "true");
+ main(conf, args);
}
private static Options createOptions() {
diff --git a/java/tools/src/java/org/apache/orc/tools/Driver.java
b/java/tools/src/java/org/apache/orc/tools/Driver.java
index 0d2778b41..c8e2fffed 100644
--- a/java/tools/src/java/org/apache/orc/tools/Driver.java
+++ b/java/tools/src/java/org/apache/orc/tools/Driver.java
@@ -102,6 +102,7 @@ public class Driver {
System.exit(1);
}
Configuration conf = new Configuration();
+ conf.set("fs.file.impl.disable.cache", "true");
Properties confSettings = options.genericOptions.getOptionProperties("D");
for(Map.Entry pair: confSettings.entrySet()) {
conf.set(pair.getKey().toString(), pair.getValue().toString());
diff --git a/java/tools/src/java/org/apache/orc/tools/FileDump.java
b/java/tools/src/java/org/apache/orc/tools/FileDump.java
index b8acb1cae..27b0e4f8a 100644
--- a/java/tools/src/java/org/apache/orc/tools/FileDump.java
+++ b/java/tools/src/java/org/apache/orc/tools/FileDump.java
@@ -142,6 +142,7 @@ public final class FileDump {
public static void main(String[] args) throws Exception {
Configuration conf = new Configuration();
+ conf.set("fs.file.impl.disable.cache", "true");
main(conf, args);
}
diff --git a/java/tools/src/java/org/apache/orc/tools/RowCount.java
b/java/tools/src/java/org/apache/orc/tools/RowCount.java
index fce0db3f4..3567f5674 100644
--- a/java/tools/src/java/org/apache/orc/tools/RowCount.java
+++ b/java/tools/src/java/org/apache/orc/tools/RowCount.java
@@ -72,7 +72,9 @@ public class RowCount {
}
public static void main(String[] args) throws Exception {
- main(new Configuration(), args);
+ Configuration conf = new Configuration();
+ conf.set("fs.file.impl.disable.cache", "true");
+ main(conf, args);
}
private static Options createOptions() {