Thanks for the update. Just one more minor comment. Would it be slightly better to also define `KafkaMetricsReporterClasses = ""` in `kafka.serverDefaults`, similar to the existing `Defaults.MetricReporterClasses = ""`? It seems that in the existing KafkaConfig.java, we always use use a default variable with value `""` rather than using `""` directly. Otherwise LGTM.
[ Full content available at: https://github.com/apache/kafka/pull/5563 ] This message was relayed via gitbox.apache.org for [email protected]
