Skip to content

Commit

Permalink
[#6375] improvment(catalog-hadoop): Remove protobuf-java to avoid c…
Browse files Browse the repository at this point in the history
…onflict with authorization module (#6376)

### 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
  • Loading branch information
yuqi1129 authored Feb 6, 2025
1 parent ca20c94 commit 4534985
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions catalogs/catalog-hadoop/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down

0 comments on commit 4534985

Please sign in to comment.