Jackie-Jiang opened a new pull request, #19985: URL: https://github.com/apache/pinot/pull/19985
## Summary `PinotMetricUtils` logs `More than one PinotMetricsFactory was found` whenever metrics factory discovery returns more than one class. That is a supported setup, not a problem: - The binary distribution ships both the `pinot-yammer` and `pinot-dropwizard` plugins. - `CompoundPinotMetricsFactory` is built to combine every discovered factory. - The factory to use is picked explicitly by the `factory.className` metrics config (e.g. `pinot.broker.metrics.factory.className`), defaulting to Yammer, so there is no ambiguity to warn about. The warning also fires in tests of any module that has both the `pinot-common` test jar (`FakeMetricsFactory`) and `pinot-yammer` on its classpath, such as `pinot-core`. This removes the warning and adds the discovered classes to the existing info log that names the selected factory, so the information is still available when the configured factory is not found. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
