chia7712 commented on PR #20405: URL: https://github.com/apache/kafka/pull/20405#issuecomment-3221510808
> Thanks for the feedback. I'm still working on adding the test case and will update the PR once it's ready. Perhaps we could enhance `testServersCanStartWithInvalidStaticConfigsAndValidDynamicConfigs` to cover this case. 1. add a custom `MetricsReporter` to the `newBroker`. the `MetricsReporter` must rely on a dynamic server configuration, such as `num.replica.fetchers` 2. add `num.replica.fetchers` to the cluster-level config 3. if `readDynamicBrokerConfigsFromSnapshot` works correctly, the custom `MetricsReporter` should not be reconfigured after a restart. -- 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
