This is an automated email from the ASF dual-hosted git repository.
jshao pushed a commit to branch branch-0.8
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/branch-0.8 by this push:
new 52d250edd7 [#6375] improvement(catalog-hadoop): Remove `protobuf-java`
to avoid conflict with authorization module (#6401)
52d250edd7 is described below
commit 52d250edd716439ce6163f074daf0d2f632a78a4
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Thu Feb 6 19:14:55 2025 +0800
[#6375] improvement(catalog-hadoop): Remove `protobuf-java` to avoid
conflict with authorization module (#6401)
### What changes were proposed in this pull request?
Remove jar `protobuf-java.jar` from the distribution package to avoid
conflicts
### Why are the changes needed?
To make authorization works for GCS fileset.
Fix: #6375
### Does this PR introduce _any_ user-facing change?
N/A.
### How was this patch tested?
N/A
Co-authored-by: Qi Yu <[email protected]>
---
catalogs/catalog-hadoop/build.gradle.kts | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/catalogs/catalog-hadoop/build.gradle.kts
b/catalogs/catalog-hadoop/build.gradle.kts
index 3108d993c1..b8877646f5 100644
--- a/catalogs/catalog-hadoop/build.gradle.kts
+++ b/catalogs/catalog-hadoop/build.gradle.kts
@@ -52,6 +52,12 @@ dependencies {
exclude("org.eclipse.jetty", "*")
exclude("io.netty")
exclude("org.fusesource.leveldbjni")
+ // Exclude `protobuf-java` 2.5.0 to avoid conflict with a higher version
of `protobuf-java`
+ // in the authorization module. The reason is that the class loader of
`catalog-hadoop` is the
+ // parent of the class loader of the authorization module, so the class
loader of `catalog-hadoop`
+ // will load the class `protobuf-java` 2.5.0 first, which will cause the
authorization module to
+ // fail to load the class `protobuf-java` 3.15.8.
+ exclude("com.google.protobuf", "protobuf-java")
}
implementation(libs.slf4j.api)
implementation(libs.awaitility)