geyanggang opened a new pull request, #11052:
URL: https://github.com/apache/gravitino/pull/11052

   
   ### What changes were proposed in this pull request?
   
   Exclude `org.apache.derby` from the Hive Metastore transitive dependencies 
in `hive-metastore2-libs` and `hive-metastore3-libs`.
   
   ### Why are the changes needed?
   
   Derby is transitively pulled into the runtime classpath of both 
hive-metastore-libs modules through Hive Metastore. It is only needed as an 
embedded database for HMS in tests — production uses MySQL/PostgreSQL. Other 
modules (`iceberg-common`, `hive-metastore-common`, `catalog-lakehouse-paimon`) 
already exclude Derby from runtime. This change makes the two libs modules 
consistent and removes an unnecessary dependency from the production 
distribution.
   
   Fix: #11051 
   
   ### Does this PR introduce _any_ user-facing change?
   
   No. Derby was never intentionally included in the production distribution 
and is not used at runtime.
   
   ### How was this patch tested?
   
   - Verified `runtimeClasspath` of both modules no longer contains Derby via 
`./gradlew :catalogs:hive-metastore2-libs:dependencies --configuration 
runtimeClasspath`
   - Build passes for both affected modules
   


-- 
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