Hi Jason, Thanks for your comments. 1) Any broker can handle this API, so admin clients will choose a node randomly. 2) I was following the RPCs for configs which support controller and broker. However, looking at all of the List… and Describe… RPCs, I see that the majority are broker-only. I have change the KIP to have only “broker” in the “listeners”.
Thanks, Andrew > On 16 Nov 2023, at 18:16, Jason Gustafson <ja...@confluent.io.INVALID> wrote: > > Hey Andrew, > > Thanks for the KIP. Just clarifying a couple small details. > > 1. I assume any broker can handle this API, so admin clients will choose a > node randomly? > 2. Does the controller need to support this API? If not, we can drop > "controller" from "listeners." > > Thanks, > Jason > > On Thu, Nov 16, 2023 at 10:00 AM Jun Rao <j...@confluent.io.invalid> wrote: > >> Hi, Andrew, >> >> Thanks for the KIP. Just one comment. >> >> Should we extend ConfigCommand or add a new tool to list client metrics? >> >> Thanks, >> >> Jun >> >> On Tue, Nov 7, 2023 at 9:42 AM Andrew Schofield < >> andrew_schofield_j...@outlook.com> wrote: >> >>> Hi, >>> I would like to start discussion of a small KIP which fills a gap in the >>> administration of client metrics configuration. >>> >>> >>> >> https://cwiki.apache.org/confluence/display/KAFKA/KIP-1000%3A+List+Client+Metrics+Configuration+Resources >>> >>> Thanks, >>> Andrew >>
