This is an automated email from the ASF dual-hosted git repository. joemcdonnell pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/impala.git
commit 52b87fcefd26dbf6fea751378bd23e0fb7e0d18a Author: Michael Smith <[email protected]> AuthorDate: Thu Sep 18 14:10:40 2025 -0700 IMPALA-14454: Exclude log4j 2 dependencies While we use reload4j, we can safely exclude log4j 2 dependencies to reduce the size of our artifacts. Change-Id: Ic060bdd969a6e5cd01646376b27c7355ce841819 Reviewed-on: http://gerrit.cloudera.org:8080/23439 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Csaba Ringhofer <[email protected]> Reviewed-by: Joe McDonnell <[email protected]> --- bin/impala-config.sh | 1 - fe/pom.xml | 22 +++++++--------------- java/executor-deps/pom.xml | 2 +- java/pom.xml | 12 ------------ 4 files changed, 8 insertions(+), 29 deletions(-) diff --git a/bin/impala-config.sh b/bin/impala-config.sh index 61b56fe2e..2718a384f 100755 --- a/bin/impala-config.sh +++ b/bin/impala-config.sh @@ -283,7 +283,6 @@ 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_ORC_JAVA_VERSION=1.7.6 export IMPALA_PAC4J_VERSION=4.5.5 export IMPALA_RELOAD4j_VERSION=1.2.22 diff --git a/fe/pom.xml b/fe/pom.xml index 846b6ca48..6b323fb89 100644 --- a/fe/pom.xml +++ b/fe/pom.xml @@ -370,16 +370,13 @@ under the License. <artifactId>*</artifactId> </exclusion> <exclusion> - <groupId>org.apache.logging.log4j</groupId> - <artifactId>log4j-slf4j-impl</artifactId> + <groupId>log4j</groupId> + <artifactId>log4j</artifactId> </exclusion> + <!-- Exclude log4j 2.x, we use reload4j --> <exclusion> <groupId>org.apache.logging.log4j</groupId> - <artifactId>log4j-1.2-api</artifactId> - </exclusion> - <exclusion> - <groupId>log4j</groupId> - <artifactId>log4j</artifactId> + <artifactId>*</artifactId> </exclusion> <!-- We should exclude hive-serde since it brings along a different version of flatbuffers causing problems for loading tables --> @@ -425,14 +422,10 @@ under the License. <artifactId>hive-standalone-metastore</artifactId> <version>${hive.version}</version> <exclusions> - <!-- Impala uses log4j v1; avoid pulling in slf4j handling for log4j2 --> - <exclusion> - <groupId>org.apache.logging.log4j</groupId> - <artifactId>log4j-slf4j-impl</artifactId> - </exclusion> + <!-- Impala uses reload4j v1; avoid pulling in log4j2 --> <exclusion> <groupId>org.apache.logging.log4j</groupId> - <artifactId>log4j-1.2-api</artifactId> + <artifactId>*</artifactId> </exclusion> <exclusion> <groupId>org.apache.hive</groupId> @@ -795,8 +788,7 @@ 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:log4j-slf4j-impl</exclude> - <exclude>org.apache.logging.log4j:log4j-1.2-api</exclude> + <exclude>org.apache.logging.log4j:*</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 dad50436e..e917c993e 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>log4j-slf4j-impl</artifactId> + <artifactId>*</artifactId> </exclusion> </exclusions> </dependency> diff --git a/java/pom.xml b/java/pom.xml index 3ae90f790..1a837e69b 100644 --- a/java/pom.xml +++ b/java/pom.xml @@ -85,7 +85,6 @@ 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,17 +382,6 @@ 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>
