xvrl commented on a change in pull request #8691: URL: https://github.com/apache/kafka/pull/8691#discussion_r428352951
########## File path: connect/mirror/src/main/java/org/apache/kafka/connect/mirror/MirrorConnectorConfig.java ########## @@ -270,9 +272,15 @@ Duration adminTimeout() { List<MetricsReporter> metricsReporters() { List<MetricsReporter> reporters = getConfiguredInstances( CommonClientConfigs.METRIC_REPORTER_CLASSES_CONFIG, MetricsReporter.class); - JmxReporter jmxReporter = new JmxReporter("kafka.connect.mirror"); + JmxReporter jmxReporter = new JmxReporter(); jmxReporter.configure(this.originals()); reporters.add(jmxReporter); + MetricsContext metricsContext = new KafkaMetricsContext("kafka.connect.mirror"); Review comment: the KIP mentions that we are deprecating the jmx prefix directly on the JmxReporter, and instead are passing it via the metrics context as the `_namespace` parameter. This doesn't change the prefix or how they are exposed in jmx. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org