chia7712 commented on PR #20938: URL: https://github.com/apache/kafka/pull/20938#issuecomment-3637249404
> And, we probably need additional tests in the code for the dynamic config update (for instance, check out DynamicConfigChangeTest) I agree. We should create a helper test to validate all dynamic configurations on the controller. We must also cover both static quorum and dynamic quorum. `ClusterTest` supports static quorum. The dynamic quorum tests could be implemented by `KafkaClusterTestKit` -- 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]
