mxm opened a new pull request, #16155: URL: https://github.com/apache/iceberg/pull/16155
Flink does not bundle `flink-metrics-dropwizard` [1][2], which enables histogram metrics. Users need to manually include the dependency for histograms to work. In Iceberg, we used to include the dependency in the flink-runtime jar. Ryan had removed the dependency in #16093 to avoid LICENSE issues, but we added it back in #16126 with the corresponding LICENSE changes, to avoid breaking changes for users. After discussing with Peter, we decided to remove the bundling going forward. This PR makes sure there are no regressions removing the dependency. I've updated the docs and the changelog to inform users of the change. References: 1. https://nightlies.apache.org/flink/flink-docs-stable/docs/ops/metrics/#histogram 2. https://github.com/apache/flink/blob/ccf45727eeb2e920f7939f453db67a411e3cb109/flink-dist/pom.xml#L284 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
