gabriellefu commented on code in PR #23121: URL: https://github.com/apache/kafka/pull/23121#discussion_r3868418767
########## core/src/main/scala/kafka/server/ConfigAdminManager.scala: ########## @@ -64,15 +65,19 @@ import scala.jdk.CollectionConverters._ * * Configuration processing is split into two parts. * - The first step, called "preprocessing," handles setting KIP-412 log levels, validating - * BROKER configurations. We also filter out some other things here like UNKNOWN resource - * types, etc. + * BROKER configurations, and performing the full GROUP configuration validation (see + * [[org.apache.kafka.coordinator.group.GroupConfig#validateOnBroker]]). We also filter out + * some other things here like UNKNOWN resource types, etc. Review Comment: thanks, I have updated -- 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]
