apoorvmittal10 commented on code in PR #14621:
URL: https://github.com/apache/kafka/pull/14621#discussion_r1378833820
##########
clients/src/main/java/org/apache/kafka/clients/admin/KafkaAdminClient.java:
##########
@@ -2619,6 +2619,9 @@ private ConfigEntry.ConfigSource
configSource(DescribeConfigsResponse.ConfigSour
case DYNAMIC_BROKER_LOGGER_CONFIG:
configSource =
ConfigEntry.ConfigSource.DYNAMIC_BROKER_LOGGER_CONFIG;
break;
+ case CLIENT_METRICS_CONFIG:
Review Comment:
You are right, this should not be needed. I have removed it, I am not sure
why other config handlers are required here when the method is just used by
topics code. May be previously it was used elsewhere as well.
--
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-unsubscr...@kafka.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
