From a305bdc03509b8b4f1303ae9817d1603957182b1 Mon Sep 17 00:00:00 2001 From: "Wang, Fei" Date: Wed, 5 Mar 2025 15:03:18 +0800 Subject: [PATCH] [KYUUBI #6960] Bump log4j version to 2.24.3 to fix ConcurrentModificationException in LoggerContext ### Why are the changes needed? Bump the log4j version to fix below issue: ``` 2025-03-04 22:27:58.291 WARN [main-SendThread(xxxx:2181)] org.apache.kyuubi.shaded.zookeeper.ClientCnxn: Session 0x0 for server null, unexpected error, closing socket connection and attempting reconnect : java.lang.ExceptionInInitializerError at org.apache.log4j.Logger.getLogger(Logger.java:35) at org.apache.kyuubi.shaded.zookeeper.Login.(Login.java:44) at org.apache.kyuubi.shaded.zookeeper.client.ZooKeeperSaslClient.createSaslClient(ZooKeeperSaslClient.java:228) at org.apache.kyuubi.shaded.zookeeper.client.ZooKeeperSaslClient.(ZooKeeperSaslClient.java:131) at org.apache.kyuubi.shaded.zookeeper.ClientCnxn$SendThread.startConnect(ClientCnxn.java:990) at org.apache.kyuubi.shaded.zookeeper.ClientCnxn$SendThread.run(ClientCnxn.java:1042) Caused by: java.util.ConcurrentModificationException at java.util.HashMap$ValueSpliterator.forEachRemaining(HashMap.java:1657) at java.util.stream.ReferencePipeline$Head.forEach(ReferencePipeline.java:647) at java.util.stream.ReferencePipeline$7$1.accept(ReferencePipeline.java:272) at java.util.WeakHashMap$ValueSpliterator.forEachRemaining(WeakHashMap.java:1216) at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:482) at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:472) at java.util.stream.ForEachOps$ForEachOp.evaluateSequential(ForEachOps.java:150) at java.util.stream.ForEachOps$ForEachOp$OfRef.evaluateSequential(ForEachOps.java:173) at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234) at java.util.stream.ReferencePipeline.forEach(ReferencePipeline.java:485) at org.apache.logging.log4j.core.LoggerContext.updateLoggers(LoggerContext.java:776) at org.apache.logging.log4j.core.LoggerContext.updateLoggers(LoggerContext.java:766) at org.apache.logging.log4j.core.config.Configurator.setLevel(Configurator.java:379) at org.apache.logging.log4j.core.config.Configurator.setLevel(Configurator.java:344) at org.apache.log4j.legacy.core.CategoryUtil.setLevel(CategoryUtil.java:131) at org.apache.log4j.Category.setLevel(Category.java:643) at org.apache.log4j.Category.setLevel(Category.java:638) at org.apache.log4j.spi.RootLogger.setLevel(RootLogger.java:60) at org.apache.log4j.spi.RootLogger.(RootLogger.java:39) at org.apache.log4j.LogManager.(LogManager.java:70) ... 6 more ``` It is fixed in https://github.com/apache/logging-log4j2/releases/tag/rel%2F2.24.3 https://github.com/apache/logging-log4j2/issues/3234 ### How was this patch tested? Existing GA. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #6960 from turboFei/log4j_version. Closes #6960 48b076c3e [Wang, Fei] Upgrade log4j version to 2.24.3 Authored-by: Wang, Fei Signed-off-by: Cheng Pan --- dev/dependencyList | 10 +++++----- pom.xml | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/dev/dependencyList b/dev/dependencyList index 6a4056125..e4c52cfa8 100644 --- a/dev/dependencyList +++ b/dev/dependencyList @@ -123,11 +123,11 @@ kubernetes-model-rbac/6.13.1//kubernetes-model-rbac-6.13.1.jar kubernetes-model-resource/6.13.1//kubernetes-model-resource-6.13.1.jar kubernetes-model-scheduling/6.13.1//kubernetes-model-scheduling-6.13.1.jar kubernetes-model-storageclass/6.13.1//kubernetes-model-storageclass-6.13.1.jar -log4j-1.2-api/2.24.2//log4j-1.2-api-2.24.2.jar -log4j-api/2.24.2//log4j-api-2.24.2.jar -log4j-core/2.24.2//log4j-core-2.24.2.jar -log4j-layout-template-json/2.24.2//log4j-layout-template-json-2.24.2.jar -log4j-slf4j-impl/2.24.2//log4j-slf4j-impl-2.24.2.jar +log4j-1.2-api/2.24.3//log4j-1.2-api-2.24.3.jar +log4j-api/2.24.3//log4j-api-2.24.3.jar +log4j-core/2.24.3//log4j-core-2.24.3.jar +log4j-layout-template-json/2.24.3//log4j-layout-template-json-2.24.3.jar +log4j-slf4j-impl/2.24.3//log4j-slf4j-impl-2.24.3.jar logging-interceptor/3.12.12//logging-interceptor-3.12.12.jar metrics-core/4.2.26//metrics-core-4.2.26.jar metrics-jmx/4.2.26//metrics-jmx-4.2.26.jar diff --git a/pom.xml b/pom.xml index 2c947bc8c..bfaa70f4d 100644 --- a/pom.xml +++ b/pom.xml @@ -177,7 +177,7 @@ 0.4.1 kyuubi-relocated-zookeeper-34 6.0.5 - 2.24.2 + 2.24.3 8.4.0 4.11.0 4.1.108.Final