hi TaiJu The current tool implementation manually constructs requests and parses responses. I think that might be the right time to refactor it to use the Admin client.
What do you think about including the following changes in this KIP? 1. Reuse the Admin client to handle request sending and response parsing, replacing the manual implementation 2. Leverage KIP-1160 to send `ApiVersionsRequest` to specific nodes 3. Add an internal `InternalDescribeFeaturesResult` to expose `ApiVersion` information. This would be used exclusively by the tool The main benefit is that the tool would be significantly streamlined, making it much easier to introduce support for the bootstrap controller Best, Chia-Ping On 2025/10/29 00:55:50 TaiJu Wu wrote: > Hi all, > > I would like to start discuss KIP-1220: kafka-broker-api-versions tool > support bootstrap controllers. > > KIP: > https://cwiki.apache.org/confluence/display/KAFKA/KIP-1220%3A+kafka-broker-api-versions+tool+support+bootstrap+controllers > JIRA: https://issues.apache.org/jira/browse/KAFKA-19663 > Draft PR: https://github.com/apache/kafka/pull/20598 > > Any feedback are appreciated. > > Best regards, > TaiJuWu >
