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

diqiu50 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 a0ecc9e99b [#10529] improvement(all): upgrade Jetty to 9.4.58 and 
exclude unused Jetty from hive-metastore3-libs (#10532)
a0ecc9e99b is described below

commit a0ecc9e99bad4764eac66c2f3a4b718983da4f37
Author: geyanggang <[email protected]>
AuthorDate: Fri Mar 27 15:16:47 2026 +0800

    [#10529] improvement(all): upgrade Jetty to 9.4.58 and exclude unused Jetty 
from hive-metastore3-libs (#10532)
    
    ### What changes were proposed in this pull request?
    
    1. Updated jetty version from 9.4.51.v20230217 to 9.4.58.v20250814 in
    `libs.versions.toml`.
    2. Added exclude("org.eclipse.jetty") to both hadoop2-common and
    hive3-metastore dependencies in `build.gradle.kts`.
    
    ### Why are the changes needed?
    
    1. The Jetty version upgrade brings the latest patches for modules that
    directly use Jetty (iceberg-rest-server, lance-rest-server).
    2. In hive-metastore3-libs, Jetty 9.3.20 is a transitive dependency from
    hadoop-common:3.1.0 (pulled in via hive3-metastore). Due to
    HiveClientClassLoader's classloader isolation, Hadoop classes are loaded
    from the base classloader which already has no Jetty, making these Jetty
    jars dead code. Both hadoop2-common and hive3-metastore need the exclude
    because Gradle resolves hadoop-common 2.10.2 → 3.1.0 (higher version
    wins), creating two dependency paths to Jetty.
    
    Fix: #10529
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    1. Verified compileJava, jar, and test pass for :catalogs:catalog-hive
    and :catalogs:hive-metastore3-libs
    2. Confirmed Jetty no longer appears in hive-metastore3-libs
    runtimeClasspath dependency tree
    
    ---------
    
    Co-authored-by: Yuhui <[email protected]>
---
 catalogs/hive-metastore3-libs/build.gradle.kts | 6 ++----
 gradle/libs.versions.toml                      | 2 +-
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/catalogs/hive-metastore3-libs/build.gradle.kts 
b/catalogs/hive-metastore3-libs/build.gradle.kts
index fa2a1a56a4..5fbae4cb36 100644
--- a/catalogs/hive-metastore3-libs/build.gradle.kts
+++ b/catalogs/hive-metastore3-libs/build.gradle.kts
@@ -36,8 +36,7 @@ dependencies {
     exclude(group = "log4j")
     exclude(group = "org.apache.avro")
     exclude(group = "org.apache.logging.log4j")
-    exclude(group = "org.eclipse.jetty.aggregate", module = "jetty-all")
-    exclude(group = "org.eclipse.jetty.orbit", module = "javax.servlet")
+    exclude(group = "org.eclipse.jetty")
     exclude(group = "org.slf4j")
   }
   implementation(libs.hive3.metastore) {
@@ -61,8 +60,7 @@ dependencies {
     exclude(group = "org.apache.logging.log4j")
     exclude(group = "org.apache.parquet", module = "parquet-hadoop-bundle")
     exclude(group = "org.datanucleus")
-    exclude(group = "org.eclipse.jetty.aggregate", module = "jetty-all")
-    exclude(group = "org.eclipse.jetty.orbit", module = "javax.servlet")
+    exclude(group = "org.eclipse.jetty")
     exclude(group = "org.openjdk.jol")
     exclude(group = "org.slf4j")
   }
diff --git a/gradle/libs.versions.toml b/gradle/libs.versions.toml
index 6e8010a77c..061a98ae1e 100644
--- a/gradle/libs.versions.toml
+++ b/gradle/libs.versions.toml
@@ -31,7 +31,7 @@ log4j = "2.24.3"
 lance = "0.39.0"
 lance-namespace = "0.0.20"
 delta-kernel = "3.3.0"
-jetty = "9.4.51.v20230217"
+jetty = "9.4.58.v20250814"
 jersey = "2.41"
 mockito = "4.11.0"
 airlift-json = "237"

Reply via email to