muralibasani commented on code in PR #23596:
URL: https://github.com/apache/kafka/pull/23596#discussion_r4114957654
##########
core/src/main/scala/kafka/server/ControllerConfigurationValidator.scala:
##########
@@ -43,7 +44,7 @@ import scala.collection.mutable
* in the same RPC, BROKER_LOGGER is not really a dynamic configuration in the
same sense
* as the others. It is not persisted to the metadata log.
*/
-class ControllerConfigurationValidator(kafkaConfig: KafkaConfig) extends
ConfigurationValidator {
+class ControllerConfigurationValidator(kafkaConfig: KafkaConfig) extends
ConfigurationValidator with Logging {
Review Comment:
If we avoid this, we need to move it to broker only
--
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]
