This is an automated email from the ASF dual-hosted git repository.
michaelsmith pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git
The following commit(s) were added to refs/heads/master by this push:
new d4992d532 Revert "IMPALA-14454: Exclude log4j 2 dependencies"
d4992d532 is described below
commit d4992d532b6fe12bce6dd12f0559b9899a547337
Author: Riza Suminto <[email protected]>
AuthorDate: Mon Dec 15 09:38:56 2025 -0800
Revert "IMPALA-14454: Exclude log4j 2 dependencies"
This reverts commit 52b87fcefd26dbf6fea751378bd23e0fb7e0d18a.
The original commit caused an issue when Impala is deployed together
with Apache Atlas. Coordinator failed to start with error message:
java.lang.NoClassDefFoundError: org/apache/logging/log4j/core/Layout
Solved minor conflict in impala-config.sh due to IMPALA-14478 applied
after IMPALA-14454.
Change-Id: I77127db8d833c675c18c30eb3d6542ca906cd2a9
Reviewed-on: http://gerrit.cloudera.org:8080/23788
Reviewed-by: Michael Smith <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
bin/impala-config.sh | 1 +
fe/pom.xml | 22 +++++++++++++++-------
java/executor-deps/pom.xml | 2 +-
java/pom.xml | 12 ++++++++++++
4 files changed, 29 insertions(+), 8 deletions(-)
diff --git a/bin/impala-config.sh b/bin/impala-config.sh
index 532735b8a..56aea598b 100755
--- a/bin/impala-config.sh
+++ b/bin/impala-config.sh
@@ -288,6 +288,7 @@ export IMPALA_JACKSON_DATABIND_VERSION=2.15.3
export IMPALA_JSON_SMART_VERSION=2.4.11
export IMPALA_JUNIT_VERSION=4.12
export IMPALA_KITE_VERSION=1.1.0
+export IMPALA_LOG4J2_VERSION=2.18.0
export IMPALA_PAC4J_VERSION=4.5.5
export IMPALA_RELOAD4j_VERSION=1.2.22
export IMPALA_SLF4J_VERSION=2.0.13
diff --git a/fe/pom.xml b/fe/pom.xml
index 248ba8ce4..ee7fe44ae 100644
--- a/fe/pom.xml
+++ b/fe/pom.xml
@@ -375,13 +375,16 @@ under the License.
<artifactId>*</artifactId>
</exclusion>
<exclusion>
- <groupId>log4j</groupId>
- <artifactId>log4j</artifactId>
+ <groupId>org.apache.logging.log4j</groupId>
+ <artifactId>log4j-slf4j-impl</artifactId>
</exclusion>
- <!-- Exclude log4j 2.x, we use reload4j -->
<exclusion>
<groupId>org.apache.logging.log4j</groupId>
- <artifactId>*</artifactId>
+ <artifactId>log4j-1.2-api</artifactId>
+ </exclusion>
+ <exclusion>
+ <groupId>log4j</groupId>
+ <artifactId>log4j</artifactId>
</exclusion>
<!-- We should exclude hive-serde since it brings along a
different version of flatbuffers causing problems for loading
tables -->
@@ -427,10 +430,14 @@ under the License.
<artifactId>hive-standalone-metastore</artifactId>
<version>${hive.version}</version>
<exclusions>
- <!-- Impala uses reload4j v1; avoid pulling in log4j2 -->
+ <!-- Impala uses log4j v1; avoid pulling in slf4j handling for log4j2
-->
<exclusion>
<groupId>org.apache.logging.log4j</groupId>
- <artifactId>*</artifactId>
+ <artifactId>log4j-slf4j-impl</artifactId>
+ </exclusion>
+ <exclusion>
+ <groupId>org.apache.logging.log4j</groupId>
+ <artifactId>log4j-1.2-api</artifactId>
</exclusion>
<exclusion>
<groupId>org.apache.hive</groupId>
@@ -912,7 +919,8 @@ under the License.
<exclude>org.apache.ant:*</exclude>
<exclude>org.eclipse.jetty:*</exclude>
<!-- We use reload4j, exclude log4j 1.x and 2.x -->
- <exclude>org.apache.logging.log4j:*</exclude>
+
<exclude>org.apache.logging.log4j:log4j-slf4j-impl</exclude>
+ <exclude>org.apache.logging.log4j:log4j-1.2-api</exclude>
<exclude>log4j:*</exclude>
<exclude>org.slf4j:slf4j-log4j12</exclude>
<!-- IMPALA-9108: Avoid pulling in leveldbjni, which is
unneeded. -->
diff --git a/java/executor-deps/pom.xml b/java/executor-deps/pom.xml
index eaace953f..1333d2abd 100644
--- a/java/executor-deps/pom.xml
+++ b/java/executor-deps/pom.xml
@@ -87,7 +87,7 @@ under the License.
</exclusion>
<exclusion>
<groupId>org.apache.logging.log4j</groupId>
- <artifactId>*</artifactId>
+ <artifactId>log4j-slf4j-impl</artifactId>
</exclusion>
</exclusions>
</dependency>
diff --git a/java/pom.xml b/java/pom.xml
index f9abb3b84..a85cd2db5 100644
--- a/java/pom.xml
+++ b/java/pom.xml
@@ -86,6 +86,7 @@ under the License.
<velocity-engine-core.version>${env.IMPALA_VELOCITY_ENGINE_CORE_VERSION}</velocity-engine-core.version>
<json-smart.version>${env.IMPALA_JSON_SMART_VERSION}</json-smart.version>
<commons-dbcp2.version>${env.IMPALA_DBCP2_VERSION}</commons-dbcp2.version>
+ <log4j2.version>${env.IMPALA_LOG4J2_VERSION}</log4j2.version>
<dropwizard-metrics.version>${env.IMPALA_DROPWIZARD_METRICS_VERSION}</dropwizard-metrics.version>
<aircompressor.version>${env.IMPALA_AIRCOMPRESSOR_VERSION}</aircompressor.version>
<datasketches.version>${env.IMPALA_DATASKETCHES_VERSION}</datasketches.version>
@@ -383,6 +384,17 @@ under the License.
<version>${jackson.version}</version>
</dependency>
+ <dependency>
+ <groupId>org.apache.logging.log4j</groupId>
+ <artifactId>log4j-api</artifactId>
+ <version>${log4j2.version}</version>
+ </dependency>
+ <dependency>
+ <groupId>org.apache.logging.log4j</groupId>
+ <artifactId>log4j-core</artifactId>
+ <version>${log4j2.version}</version>
+ </dependency>
+
<dependency>
<groupId>org.apache.datasketches</groupId>
<artifactId>datasketches-java</artifactId>
