ericyuan915 opened a new pull request, #18982:
URL: https://github.com/apache/hudi/pull/18982
### Change Logs
`hudi-flink-bundle` bundles `org.apache.flink:flink-metrics-dropwizard` and
(since #18730) relocates `com.codahale.metrics` →
`org.apache.hudi.com.codahale.metrics`, but does not relocate
`org.apache.flink.dropwizard`. maven-shade recompiles the bundled
`DropwizardHistogramWrapper` against the relocated codahale while leaving it at
the real Flink FQN, so the shaded jar exposes a `DropwizardHistogramWrapper`
whose only constructor is `(org.apache.hudi.com.codahale.metrics.Histogram)`.
Any app with both `hudi-flink-bundle` and the real `flink-metrics-dropwizard`
on the classpath then hits `NoSuchMethodError` on `new
DropwizardHistogramWrapper(new com.codahale.metrics.Histogram(...))`.
This PR relocates `org.apache.flink.dropwizard` alongside codahale so the
bundled bridge moves into the hudi namespace; shade rewrites Hudi's own
references and the genuine Flink FQN is left intact for other consumers.
Fixes #<[18981](https://github.com/apache/hudi/issues/18981)>
### Impact
Fixes a runtime `NoSuchMethodError` for jobs combining `hudi-flink-bundle`
with `flink-metrics-dropwizard`. No behavior change for Hudi's own metrics.
### Risk level: low
Shade-config-only change, scoped to `hudi-flink-bundle`.
### Documentation Update
None.
### Contributor's checklist
- [x] Read through the [contributor's
guide](https://hudi.apache.org/contribute/how-to-contribute)
- [x] Change Logs and Impact were stated clearly
- [x] Adequate tests were added if applicable (N/A — shade-config only)
- [x] CI passed
--
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]
