[GitHub] [kafka] dielhennr commented on a change in pull request #10772: KAFKA-12697: Add FencedBrokerCount and ActiveBrokerCount metrics to the QuorumController

[GitBox <gi...@apache.org>]

dielhennr commented on a change in pull request #10772:
URL: https://github.com/apache/kafka/pull/10772#discussion_r732112282



##########
File path: metadata/src/main/java/org/apache/kafka/controller/QuorumControllerMetrics.java
##########
@@ -26,28 +26,36 @@
 import java.util.Objects;
 
 public final class QuorumControllerMetrics implements ControllerMetrics {
-    private final static MetricName ACTIVE_CONTROLLER_COUNT = getMetricName(
-        "KafkaController", "ActiveControllerCount");
-    private final static MetricName EVENT_QUEUE_TIME_MS = getMetricName(
-        "ControllerEventManager", "EventQueueTimeMs");
-    private final static MetricName EVENT_QUEUE_PROCESSING_TIME_MS = getMetricName(
-        "ControllerEventManager", "EventQueueProcessingTimeMs");
-    private final static MetricName GLOBAL_TOPIC_COUNT = getMetricName(
-        "KafkaController", "GlobalTopicCount");
-    private final static MetricName GLOBAL_PARTITION_COUNT = getMetricName(
-        "KafkaController", "GlobalPartitionCount");
-    private final static MetricName OFFLINE_PARTITION_COUNT = getMetricName(
-        "KafkaController", "OfflinePartitionsCount");
-    private final static MetricName PREFERRED_REPLICA_IMBALANCE_COUNT = getMetricName(
-        "KafkaController", "PreferredReplicaImbalanceCount");
-
+    private final static MetricName ACTIVE_CONTROLLER_COUNT = new MetricName(

Review comment:
       This was a mistake when I merged trunk into my branch. I reverted it.




-- 
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: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org