chia7712 commented on PR #15645: URL: https://github.com/apache/kafka/pull/15645#issuecomment-2047302613
> I thought we want to move all code related to the ConfigCommand to java and remove it when ZK support will be dropped. Personally, rewriting a class which will get removed totally is a bit weird. However, I understand your purpose and effort of #15417, and hence could we make this PR more valuable by increasing its coverage? I mean we can add more tests for zk and kraft in this PR, and it would be nice to use new test infra (`ClusterTestExtensions`) to rewrite it. WDYT? -- 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
