(gravitino) branch main updated: [#6991] fix(gvfs): Exclude log4j in gvfs runtime jar (#6992)

[jshao]

This is an automated email from the ASF dual-hosted git repository.

jshao pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new 83fe8a0a2d [#6991] fix(gvfs): Exclude log4j in gvfs runtime jar (#6992)
83fe8a0a2d is described below

commit 83fe8a0a2dd97b24852cc43b3a538dbe7e450bc6
Author: mchades <liminghu...@datastrato.com>
AuthorDate: Thu Apr 17 18:13:30 2025 +0800

    [#6991] fix(gvfs): Exclude log4j in gvfs runtime jar (#6992)
    
    ### What changes were proposed in this pull request?
    
    Exclude the log4j jar in gvfs runtime shading jar to fix the log
    problem.
    
    ### Why are the changes needed?
    
    The log4j jar shipped with GVFS runtime jar will have a conflict with
    the environment, which will make the log4j not work, so we should
    exclude the log4j jar in GVFS runtime jar.
    
    Fix: #6991
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Existing tests.
    
    Co-authored-by: Jerry Shao <jerrys...@datastrato.com>
---
 api/build.gradle.kts                                | 1 -
 clients/client-java/build.gradle.kts                | 4 +++-
 clients/filesystem-hadoop3-runtime/build.gradle.kts | 4 +++-
 clients/filesystem-hadoop3/build.gradle.kts         | 3 ++-
 4 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/api/build.gradle.kts b/api/build.gradle.kts
index b13745d9ff..b4399b13c0 100644
--- a/api/build.gradle.kts
+++ b/api/build.gradle.kts
@@ -26,7 +26,6 @@ dependencies {
   implementation(libs.commons.lang3)
   implementation(libs.commons.collections4)
   implementation(libs.guava)
-  implementation(libs.slf4j.api)
 
   testImplementation(libs.junit.jupiter.api)
   testImplementation(libs.junit.jupiter.params)
diff --git a/clients/client-java/build.gradle.kts 
b/clients/client-java/build.gradle.kts
index d7518569c9..9d1681645b 100644
--- a/clients/client-java/build.gradle.kts
+++ b/clients/client-java/build.gradle.kts
@@ -30,7 +30,9 @@ dependencies {
   implementation(libs.jackson.datatype.jdk8)
   implementation(libs.jackson.datatype.jsr310)
   implementation(libs.guava)
-  implementation(libs.httpclient5)
+  implementation(libs.httpclient5) {
+    exclude(group = "org.slf4j")
+  }
   implementation(libs.commons.lang3)
 
   compileOnly(libs.lombok)
diff --git a/clients/filesystem-hadoop3-runtime/build.gradle.kts 
b/clients/filesystem-hadoop3-runtime/build.gradle.kts
index 38e8e162be..021c246cc4 100644
--- a/clients/filesystem-hadoop3-runtime/build.gradle.kts
+++ b/clients/filesystem-hadoop3-runtime/build.gradle.kts
@@ -26,7 +26,9 @@ plugins {
 }
 
 dependencies {
-  implementation(project(":clients:filesystem-hadoop3"))
+  implementation(project(":clients:filesystem-hadoop3")) {
+    exclude(group = "org.slf4j")
+  }
   implementation(project(":clients:client-java-runtime", configuration = 
"shadow"))
   implementation(libs.commons.lang3)
 }
diff --git a/clients/filesystem-hadoop3/build.gradle.kts 
b/clients/filesystem-hadoop3/build.gradle.kts
index 424f6a1140..5570f14db2 100644
--- a/clients/filesystem-hadoop3/build.gradle.kts
+++ b/clients/filesystem-hadoop3/build.gradle.kts
@@ -36,8 +36,9 @@ dependencies {
   }
 
   implementation(libs.caffeine)
-  implementation(libs.guava)
   implementation(libs.commons.lang3)
+  implementation(libs.guava)
+  implementation(libs.slf4j.api)
 
   testImplementation(project(":api"))
   testImplementation(project(":core"))