omkreddy commented on pull request #8715: URL: https://github.com/apache/kafka/pull/8715#issuecomment-639031210
@gnkoshelev Thanks for the PR. LGTM. I think, we can merge both PRs (#8717 ). `AdminZkClient.validateTopicConfig()` is used in command line tools (This will go when remove ZK dependency). we may want throw same error in all places. ---------------------------------------------------------------- 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
