mingyen066 opened a new pull request, #17442: URL: https://github.com/apache/kafka/pull/17442
Originally, there were no unit tests in the `transaction-coordinator` module, which made it easy to introduce errors when modifying the code. There are some fields are dynamic, which means we need to make sure all values are directly get from Abstract config so that if [Kafka config update its config](https://github.com/apache/kafka/blob/9b62c861fa4650e58a393384ed16dcec9df32fc5/core/src/main/scala/kafka/server/KafkaConfig.scala#L190-L196), the program can get latest value. ### Changes: * Add UT to make sure all declared config are defined in `CONFIG_DEF` * Add UT to make sure static config are retrieve from class attribute and dynamic values are retrieved from `AbstractConfig` ### Committer Checklist (excluded from commit message) - [ ] Verify design and implementation - [ ] Verify test coverage and CI build status - [ ] Verify documentation (including upgrade notes) -- 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]
