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 159bb023b [#4678] improve(auth-ranger): Shrink Ranger authorization 
plugin package size (#4756)
159bb023b is described below

commit 159bb023b4c1102671368c0709fb9a181bb7ed12
Author: Xun <[email protected]>
AuthorDate: Thu Aug 29 12:35:19 2024 +0800

    [#4678] improve(auth-ranger): Shrink Ranger authorization plugin package 
size (#4756)
    
    ### What changes were proposed in this pull request?
    
    Exclude invalidate depends on JAR package in the Rangere authorization
    module.
    
    ```
    -rw-r--r--  1 xun  staff   657952 Aug 29 11:27 commons-lang3-3.14.0.jar
    -rw-r--r--  1 xun  staff   238400 Aug 29 11:27 commons-text-1.10.0.jar
    -rw-r--r--  1 xun  staff  1621691 Aug 29 11:27 jersey-bundle-1.19.3.jar
    -rw-r--r--  1 xun  staff   780664 Aug 29 11:27 jackson-mapper-asl-1.9.13.jar
    -rw-r--r--  1 xun  staff    19936 Aug 29 11:27 jsr305-3.0.2.jar
    -rw-r--r--  1 xun  staff   284220 Aug 29 11:27 commons-lang-2.6.jar
    -rw-r--r--  1 xun  staff     4112 Aug 29 11:27 gethostname4j-1.0.0.jar
    -rw-r--r--  1 xun  staff   126898 Aug 29 11:27 javax.ws.rs-api-2.1.1.jar
    -rw-r--r--  1 xun  staff    16830 Aug 29 11:27 
error_prone_annotations-2.21.1.jar
    -rw-r--r--  1 xun  staff     2199 Aug 29 11:27 
listenablefuture-9999.0-empty-to-avoid-conflict-with-guava.jar
    -rw-r--r--  1 xun  staff   232248 Aug 29 11:27 jackson-core-asl-1.9.13.jar
    -rw-r--r--  1 xun  staff   224460 Aug 29 11:27 checker-qual-3.37.0.jar
    -rw-r--r--  1 xun  staff    34760 Aug 29 11:27 
gravitino-authorization-ranger-0.7.0-incubating-SNAPSHOT.jar
    -rw-r--r--  1 xun  staff   128076 Aug 29 11:27 jaxb-api-2.3.1.jar
    -rw-r--r--  1 xun  staff  1055245 Aug 29 11:27 
ranger-plugins-common-2.4.0.jar
    -rw-r--r--  1 xun  staff     4617 Aug 29 11:27 failureaccess-1.0.1.jar
    -rw-r--r--  1 xun  staff    30299 Aug 29 11:27 ranger-intg-2.4.0.jar
    -rw-r--r--  1 xun  staff   700226 Aug 29 11:27 
gravitino-core-0.7.0-incubating-SNAPSHOT.jar
    -rw-r--r--  1 xun  staff    18336 Aug 29 11:27 jackson-jaxrs-1.9.13.jar
    -rw-r--r--  1 xun  staff   632505 Aug 29 11:27 
commons-configuration2-2.8.0.jar
    -rw-r--r--  1 xun  staff    61829 Aug 29 11:27 commons-logging-1.2.jar
    -rw-r--r--  1 xun  staff   229171 Aug 29 11:27 
gravitino-api-0.7.0-incubating-SNAPSHOT.jar
    ```
    
    ### Why are the changes needed?
    
    Reduce Rangere authorization library package size from `16.7 MB` to `7.1
    MB`
    
    Fix: #4678
    
    ### Does this PR introduce _any_ user-facing change?
    
    N/A
    
    ### How was this patch tested?
    
    CI passed.
---
 authorizations/authorization-ranger/build.gradle.kts | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/authorizations/authorization-ranger/build.gradle.kts 
b/authorizations/authorization-ranger/build.gradle.kts
index 9121a414a..51dc592a7 100644
--- a/authorizations/authorization-ranger/build.gradle.kts
+++ b/authorizations/authorization-ranger/build.gradle.kts
@@ -32,13 +32,10 @@ dependencies {
     exclude(group = "*")
   }
   implementation(libs.bundles.log4j)
-  implementation(libs.commons.collections4)
   implementation(libs.commons.lang3)
   implementation(libs.guava)
-  implementation(libs.slf4j.api)
 
   compileOnly(libs.lombok)
-  implementation(libs.jackson.annotations)
   implementation(libs.ranger.intg) {
     exclude("org.apache.hadoop", "hadoop-common")
     exclude("org.apache.hive", "hive-storage-api")
@@ -50,7 +47,8 @@ dependencies {
     exclude("org.elasticsearch.plugin")
     exclude("org.apache.ranger", "ranger-plugins-audit")
     exclude("org.apache.ranger", "ranger-plugins-cred")
-    exclude("org.apache.ranger", "ranger-plugins-classloader")
+    exclude("org.apache.ranger", "ranger-plugin-classloader")
+    exclude("net.java.dev.jna")
     exclude("javax.ws.rs")
   }
   implementation(libs.javax.ws.rs.api)
@@ -76,6 +74,7 @@ dependencies {
     exclude("org.elasticsearch.client")
     exclude("org.elasticsearch.plugin")
     exclude("javax.ws.rs")
+    exclude("org.apache.ranger", "ranger-plugin-classloader")
   }
   testImplementation(libs.hive2.jdbc) {
     exclude("org.slf4j")
@@ -91,7 +90,11 @@ tasks {
 
   val copyAuthorizationLibs by registering(Copy::class) {
     dependsOn("jar", "runtimeJars")
-    from("build/libs")
+    from("build/libs") {
+      exclude("guava-*.jar")
+      exclude("log4j-*.jar")
+      exclude("slf4j-*.jar")
+    }
     into("$rootDir/distribution/package/authorizations/ranger/libs")
   }
 

Reply via email to