From dc4265884b62199732b33346057388411f85b753 Mon Sep 17 00:00:00 2001
From: zombee0 <ewang2027@gmail.com>
Date: Thu, 17 Apr 2025 19:26:37 +0800
Subject: [PATCH] [BugFix] use log4j2 as implementation of slf4j (#58005)

Signed-off-by: zombee0 <ewang2027@gmail.com>
---
 java-extensions/hadoop-lib/pom.xml            |  5 +++
 .../utils/loader/ChildFirstClassLoader.java   |  4 +-
 java-extensions/pom.xml                       | 37 +++++++++++++++++++
 3 files changed, 45 insertions(+), 1 deletion(-)

diff --git a/java-extensions/hadoop-lib/pom.xml b/java-extensions/hadoop-lib/pom.xml
index ca08e648dd2..53f88bd3853 100644
--- a/java-extensions/hadoop-lib/pom.xml
+++ b/java-extensions/hadoop-lib/pom.xml
@@ -17,6 +17,11 @@
     </properties>
 
     <dependencies>
+        <dependency>
+            <groupId>org.apache.logging.log4j</groupId>
+            <artifactId>log4j-slf4j-impl</artifactId>
+        </dependency>
+
         <dependency>
             <groupId>org.apache.hadoop</groupId>
             <artifactId>hadoop-aws</artifactId>
diff --git a/java-extensions/java-utils/src/main/java/com/starrocks/utils/loader/ChildFirstClassLoader.java b/java-extensions/java-utils/src/main/java/com/starrocks/utils/loader/ChildFirstClassLoader.java
index 7f8cc997e08..0369b9c9659 100644
--- a/java-extensions/java-utils/src/main/java/com/starrocks/utils/loader/ChildFirstClassLoader.java
+++ b/java-extensions/java-utils/src/main/java/com/starrocks/utils/loader/ChildFirstClassLoader.java
@@ -18,6 +18,7 @@ import java.io.IOException;
 import java.net.URL;
 import java.net.URLClassLoader;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.Collections;
 import java.util.Enumeration;
 
@@ -38,7 +39,8 @@ public class ChildFirstClassLoader extends URLClassLoader {
         super(urls, null);
         this.parentLoader = new ParentClassLoader(parent);
         // load native method class from parent
-        this.parentFirstClass = new ArrayList<>(Collections.singleton("com.starrocks.utils.NativeMethodHelper"));
+        this.parentFirstClass = new ArrayList<>(
+                Arrays.asList("com.starrocks.utils.NativeMethodHelper", "org.slf4j.ILoggerFactory", "org.slf4j.Logger"));
     }
 
     @Override
diff --git a/java-extensions/pom.xml b/java-extensions/pom.xml
index 2ec4ab5208e..35c60c24bb6 100644
--- a/java-extensions/pom.xml
+++ b/java-extensions/pom.xml
@@ -132,6 +132,10 @@
                         <groupId>org.apache.zookeeper</groupId>
                         <artifactId>zookeeper</artifactId>
                     </exclusion>
+                    <exclusion>
+                        <groupId>org.slf4j</groupId>
+                        <artifactId>slf4j-reload4j</artifactId>
+                    </exclusion>
                     <exclusion>
                         <groupId>org.apache.avro</groupId>
                         <artifactId>avro</artifactId>
@@ -185,6 +189,10 @@
                         <groupId>com.google.protobuf</groupId>
                         <artifactId>protobuf-java</artifactId>
                     </exclusion>
+                    <exclusion>
+                        <groupId>org.slf4j</groupId>
+                        <artifactId>slf4j-reload4j</artifactId>
+                    </exclusion>
                 </exclusions>
             </dependency>
 
@@ -192,6 +200,12 @@
                 <groupId>org.apache.hadoop</groupId>
                 <artifactId>hadoop-kms</artifactId>
                 <version>${hadoop.version}</version>
+                <exclusions>
+                    <exclusion>
+                        <groupId>org.slf4j</groupId>
+                        <artifactId>slf4j-reload4j</artifactId>
+                    </exclusion>
+                </exclusions>
             </dependency>
 
             <dependency>
@@ -204,18 +218,36 @@
                 <groupId>org.apache.hadoop</groupId>
                 <artifactId>hadoop-hdfs-httpfs</artifactId>
                 <version>${hadoop.version}</version>
+                <exclusions>
+                    <exclusion>
+                        <groupId>org.slf4j</groupId>
+                        <artifactId>slf4j-reload4j</artifactId>
+                    </exclusion>
+                </exclusions>
             </dependency>
 
             <dependency>
                 <groupId>org.apache.hadoop</groupId>
                 <artifactId>hadoop-nfs</artifactId>
                 <version>${hadoop.version}</version>
+                <exclusions>
+                    <exclusion>
+                        <groupId>org.slf4j</groupId>
+                        <artifactId>slf4j-reload4j</artifactId>
+                    </exclusion>
+                </exclusions>
             </dependency>
 
             <dependency>
                 <groupId>org.apache.hadoop</groupId>
                 <artifactId>hadoop-hdfs-nfs</artifactId>
                 <version>${hadoop.version}</version>
+                <exclusions>
+                    <exclusion>
+                        <groupId>org.slf4j</groupId>
+                        <artifactId>slf4j-reload4j</artifactId>
+                    </exclusion>
+                </exclusions>
             </dependency>
 
             <dependency>
@@ -295,6 +327,11 @@
                 <artifactId>log4j-core</artifactId>
                 <version>${log4j2.version}</version>
             </dependency>
+            <dependency>
+                <groupId>org.apache.logging.log4j</groupId>
+                <artifactId>log4j-slf4j-impl</artifactId>
+                <version>${log4j2.version}</version>
+            </dependency>
             <!--log4j 2-->
 
             <!-- junit -->