This is an automated email from the ASF dual-hosted git repository.
chengpan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kyuubi.git
The following commit(s) were added to refs/heads/master by this push:
new a305bdc035 [KYUUBI #6960] Bump log4j version to 2.24.3 to fix
ConcurrentModificationException in LoggerContext
a305bdc035 is described below
commit a305bdc03509b8b4f1303ae9817d1603957182b1
Author: Wang, Fei <[email protected]>
AuthorDate: Wed Mar 5 15:03:18 2025 +0800
[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.<init>(Login.java:44)
at
org.apache.kyuubi.shaded.zookeeper.client.ZooKeeperSaslClient.createSaslClient(ZooKeeperSaslClient.java:228)
at
org.apache.kyuubi.shaded.zookeeper.client.ZooKeeperSaslClient.<init>(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.<init>(RootLogger.java:39)
at org.apache.log4j.LogManager.<clinit>(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 <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
---
dev/dependencyList | 10 +++++-----
pom.xml | 2 +-
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/dev/dependencyList b/dev/dependencyList
index 6a4056125f..e4c52cfa81 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 2c947bc8c0..bfaa70f4dc 100644
--- a/pom.xml
+++ b/pom.xml
@@ -177,7 +177,7 @@
<kyuubi-relocated.version>0.4.1</kyuubi-relocated.version>
<kyuubi-relocated-zookeeper.artifacts>kyuubi-relocated-zookeeper-34</kyuubi-relocated-zookeeper.artifacts>
<ldapsdk.version>6.0.5</ldapsdk.version>
- <log4j.version>2.24.2</log4j.version>
+ <log4j.version>2.24.3</log4j.version>
<mysql.jdbc.version>8.4.0</mysql.jdbc.version>
<mockito.version>4.11.0</mockito.version>
<netty.version>4.1.108.Final</netty.version>
