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

   
   
   ### What changes were proposed in this pull request?
   
   Remove the `htrace-core4-4.1.0-incubating` dependency from the project. This 
library is no longer needed as Hadoop 3.3.x does not require it at runtime.
   Changes:
   - Remove `htrace-core4` version and library declaration from 
libs.versions.toml
   - Remove `implementation(libs.htrace.core4)` from `catalog-hive` and 
`catalog-lakehouse-hudi`
   - Remove `testImplementation(libs.htrace.core4)` from 
`catalog-lakehouse-hudi` and `hive-metastore-common`
   
   ### Why are the changes needed?
   
   Apache HTrace has been retired (moved to Apache Attic) and is no longer 
maintained. Hadoop 3.3.x rewrote its tracing subsystem and no longer depends on 
htrace-core4 at runtime. The existing cleanup code in 
`ClassLoaderResourceCleanerUtils` already handles the absence of htrace 
gracefully via reflection with try-catch. Removing this unused dependency 
reduces the project's dependency footprint.
   
   Fix: #11314 
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   - Verified compilation of all affected modules (`catalog-hive`, 
`catalog-lakehouse-hudi`, `hive-metastore-common`)
   - Ran unit tests for all affected modules — all passed
   - Verified `htrace-core4` jar is no longer present in distribution output
   - Confirmed no source code imports from `org.apache.htrace` (only 
reflection-based reference in `ClassLoaderResourceCleanerUtils` which is 
already guarded by try-catch)
   


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