rangareddy commented on issue #17016:
URL: https://github.com/apache/hudi/issues/17016#issuecomment-5391445399

   This issue was reviewed as part of the JIRA-migrated backlog triage 
(HUDI-9416).
   
   **Findings: still open.**
   
   Not done: `aircompressor` is still a plain dependency at 
`hudi-io/pom.xml:154`, with no shade relocation configured for it.
   
   Usefully, the description already answers half the question: shading is 
**not** required for Trino, because the SPI guarantees classloader isolation 
(the plugin spec is quoted in the body). So the open scope is the Presto side 
only, which is worth reflecting in the title.
   
   One current data point that supports keeping this on the radar: 
`packaging/hudi-integ-test-bundle/pom.xml:300` carries a note that `trino-jdbc` 
bundles aircompressor v3 compiled to a newer class-file version - so version 
skew on this artifact is already causing real friction elsewhere in the build.
   
   Keeping this open.
   


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