geyanggang opened a new pull request, #10532:
URL: https://github.com/apache/gravitino/pull/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
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to