Hi Andrew, Thanks for looking into this PR!
AS1: Agree, we could also use DESCRIBE_CLUSTER for broker, I will change the implementation after this KIP is accepted, but this change is implementation detail for old implementation and not related to the new public api so I do not update KIP. Thanks, TaiJuWu Andrew Schofield <[email protected]> 於 2025年12月3日週三 上午1:32寫道： > Hi TaiWuJu, > Thanks for the KIP. I think this is very helpful for Kafka developers > working > with the Kafka protocol. > > AS1: Trying out the PR, I notice that the tool uses DESCRIBE_CLUSTER > to talk to controllers, but it uses METADATA to talk to brokers. Since > both brokers and controllers support DESCRIBE_CLUSTER in recent > releases, I wonder whether you could also use DESCRIBE_CLUSTER > If available with brokers too. The advantage is that the error handling > when using --bootstrap-server with a controller is much nicer. > > Thanks, > Andrew > > > On 2 Dec 2025, at 04:35, TaiJu Wu <[email protected]> wrote: > > > > Hi Kirk, > > > > KT1. fixed. Thanks your point out. > > KT2. just like with other tools, --bootstrap-controller and > > --bootstrap-server cannot be used together in the same invocation. They > are > > mutually exclusive options and the user must choose one base on their > > target. > > > > Best Regards, > > TaiJuWu > > > > Kirk True <[email protected]> 於 2025年12月2日週二 上午8:29寫道： > > > >> Hi TaiJu, > >> > >> Thanks for the KIP! > >> > >> Some minor points... > >> > >> KT1. I think the second sentence should use "broker's" instead of > >> "controller's". > >> > >> KT2. Can --bootstrap-controller and --bootstrap-server be used in the > same > >> invocation of the tool? > >> > >> Thanks, > >> Kirk > >> > >> On Sun, Nov 30, 2025, at 5:48 PM, TaiJu Wu wrote: > >>> Hi all, > >>> > >>> I would like to bump this thread for discussion. > >>> > >>> TaiJu Wu <[email protected]> 於 2025年10月29日週三 上午8:55寫道： > >>> > >>>> 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 > >>>> > >>> > >> > >
