This is an automated email from the ASF dual-hosted git repository.
Jackie-Jiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new e8647425026 Remove the warning on multiple PinotMetricsFactory classes
(#19985)
e8647425026 is described below
commit e8647425026d9d490112b28fe22f5a7a44586a77
Author: Xiaotian (Jackie) Jiang <[email protected]>
AuthorDate: Tue Oct 20 21:38:56 2026 -0700
Remove the warning on multiple PinotMetricsFactory classes (#19985)
---
.../main/java/org/apache/pinot/spi/metrics/PinotMetricUtils.java | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git
a/pinot-spi/src/main/java/org/apache/pinot/spi/metrics/PinotMetricUtils.java
b/pinot-spi/src/main/java/org/apache/pinot/spi/metrics/PinotMetricUtils.java
index 8d5b5841967..b98bfca6ae6 100644
--- a/pinot-spi/src/main/java/org/apache/pinot/spi/metrics/PinotMetricUtils.java
+++ b/pinot-spi/src/main/java/org/apache/pinot/spi/metrics/PinotMetricUtils.java
@@ -68,13 +68,10 @@ public class PinotMetricUtils {
/// @param metricsConfiguration The subset of the configuration containing
the metrics-related keys
private static void initializePinotMetricsFactory(PinotConfiguration
metricsConfiguration) {
Set<Class<?>> classes = getPinotMetricsFactoryClasses();
- if (classes.size() > 1) {
- LOGGER.warn("More than one PinotMetricsFactory was found: {}", classes);
- }
-
String metricsFactoryClassName =
metricsConfiguration.getProperty(CONFIG_OF_METRICS_FACTORY_CLASS_NAME,
DEFAULT_METRICS_FACTORY_CLASS_NAME);
- LOGGER.info("{} will be initialized as the PinotMetricsFactory",
metricsFactoryClassName);
+ LOGGER.info("{} will be initialized as the PinotMetricsFactory out of the
found classes: {}",
+ metricsFactoryClassName, classes);
Optional<Class<?>> clazzFound = classes.stream().filter(c ->
c.getName().equals(metricsFactoryClassName))
.findFirst();
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
