tombentley commented on pull request #9847: URL: https://github.com/apache/kafka/pull/9847#issuecomment-784925092
Rebased for conflict. @rondagostino I don't quite understand the plan for how topic configs are going to work with KIP-500. The `ConfigHelper` seems to provide an abstraction for `DescribeConfigs` RPC, but it seems that currently `AlterConfigs` and `IncrementalAlterConfigs` require ZK. Presumably an abstraction for these is in the works, and that would also need to handle default topic configs in a consistent way to what I'm trying to do here. I need to ensure that the ZKless path also has test coverage. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
