This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch branch-1.9
in repository https://gitbox.apache.org/repos/asf/orc.git
The following commit(s) were added to refs/heads/branch-1.9 by this push:
new 1f78ee450 ORC-1909: Remove unused test resource `log4j.properties`
files
1f78ee450 is described below
commit 1f78ee4506add0e598e3938adb543843d4a32c55
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Sun Jun 8 18:24:39 2025 -0700
ORC-1909: Remove unused test resource `log4j.properties` files
### What changes were proposed in this pull request?
This PR aims to remove unused test resource `log4j.properties` files.
### Why are the changes needed?
Aapche ORC 1.7.5 switched the deprecated `slf4j-log4j12` to `slf4j-simple`
via ORC-1182. So, these test resources are unused in ORC 1.7.6+.
```
$ find . -name log4j.properties
./java/bench/core/src/resources/log4j.properties
./java/tools/src/test/resources/log4j.properties
./java/core/src/test/resources/log4j.properties
./java/shims/src/test/resources/log4j.properties
./java/mapreduce/src/test/resources/log4j.properties
```
### How was this patch tested?
Pass the CIs and manual review because this doesn't affect ORC code.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #2253 from dongjoon-hyun/ORC-1909.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
(cherry picked from commit 4431151c98b23008edb2828f21dd3a2e494ec41e)
Signed-off-by: Dongjoon Hyun <[email protected]>
---
java/bench/core/src/resources/log4j.properties | 18 ------------------
java/core/src/test/resources/log4j.properties | 20 --------------------
java/mapreduce/src/test/resources/log4j.properties | 20 --------------------
java/shims/src/test/resources/log4j.properties | 20 --------------------
java/tools/src/test/resources/log4j.properties | 21 ---------------------
5 files changed, 99 deletions(-)
diff --git a/java/bench/core/src/resources/log4j.properties
b/java/bench/core/src/resources/log4j.properties
deleted file mode 100644
index 0df3f70e5..000000000
--- a/java/bench/core/src/resources/log4j.properties
+++ /dev/null
@@ -1,18 +0,0 @@
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-
-log4j.rootLogger=WARN, CONSOLE
-
-# CONSOLE is set to be a ConsoleAppender using a PatternLayout
-log4j.appender.CONSOLE=org.apache.log4j.ConsoleAppender
-log4j.appender.CONSOLE.layout=org.apache.log4j.PatternLayout
-log4j.appender.CONSOLE.layout.ConversionPattern=[%-5p] %m%n
diff --git a/java/core/src/test/resources/log4j.properties
b/java/core/src/test/resources/log4j.properties
deleted file mode 100644
index 3979ce078..000000000
--- a/java/core/src/test/resources/log4j.properties
+++ /dev/null
@@ -1,20 +0,0 @@
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-
-log4j.rootLogger=WARN,stdout
-
-log4j.appender.stdout=org.apache.log4j.ConsoleAppender
-log4j.appender.stdout.layout=org.apache.log4j.PatternLayout
-log4j.appender.stdout.layout.ConversionPattern=%p\t%d{ISO8601}\t%r\t%c\t[%t]\t%m%n
-
-# Suppress the warnings about native io not being available
-log4j.logger.org.apache.hadoop.util.NativeCodeLoader=ERROR
diff --git a/java/mapreduce/src/test/resources/log4j.properties
b/java/mapreduce/src/test/resources/log4j.properties
deleted file mode 100644
index 3979ce078..000000000
--- a/java/mapreduce/src/test/resources/log4j.properties
+++ /dev/null
@@ -1,20 +0,0 @@
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-
-log4j.rootLogger=WARN,stdout
-
-log4j.appender.stdout=org.apache.log4j.ConsoleAppender
-log4j.appender.stdout.layout=org.apache.log4j.PatternLayout
-log4j.appender.stdout.layout.ConversionPattern=%p\t%d{ISO8601}\t%r\t%c\t[%t]\t%m%n
-
-# Suppress the warnings about native io not being available
-log4j.logger.org.apache.hadoop.util.NativeCodeLoader=ERROR
diff --git a/java/shims/src/test/resources/log4j.properties
b/java/shims/src/test/resources/log4j.properties
deleted file mode 100644
index 3979ce078..000000000
--- a/java/shims/src/test/resources/log4j.properties
+++ /dev/null
@@ -1,20 +0,0 @@
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-
-log4j.rootLogger=WARN,stdout
-
-log4j.appender.stdout=org.apache.log4j.ConsoleAppender
-log4j.appender.stdout.layout=org.apache.log4j.PatternLayout
-log4j.appender.stdout.layout.ConversionPattern=%p\t%d{ISO8601}\t%r\t%c\t[%t]\t%m%n
-
-# Suppress the warnings about native io not being available
-log4j.logger.org.apache.hadoop.util.NativeCodeLoader=ERROR
diff --git a/java/tools/src/test/resources/log4j.properties
b/java/tools/src/test/resources/log4j.properties
deleted file mode 100644
index 8feed70c2..000000000
--- a/java/tools/src/test/resources/log4j.properties
+++ /dev/null
@@ -1,21 +0,0 @@
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-
-log4j.rootLogger=WARN,stdout
-
-log4j.appender.stdout=org.apache.log4j.ConsoleAppender
-log4j.appender.stdout.Target = System.err
-log4j.appender.stdout.layout=org.apache.log4j.PatternLayout
-log4j.appender.stdout.layout.ConversionPattern=%p\t%d{ISO8601}\t%r\t%c\t[%t]\t%m%n
-
-# Suppress the warnings about native io not being available
-log4j.logger.org.apache.hadoop.util.NativeCodeLoader=ERROR
