Hello all, Thank you all for the votes! With 3 binding and 1 non-binding vote this KIP will be accepted.
Regards, Ryan Dielhenn On Tue, Sep 21, 2021 at 11:44 AM Ismael Juma <ism...@juma.me.uk> wrote: > Thanks for the KIP, +1 (binding). > > Ismael > > On Thu, Sep 2, 2021 at 1:20 PM Ryan Dielhenn <rdielh...@confluent.io > .invalid> > wrote: > > > Hello kafka devs, > > > > I would like to start a vote on KIP-771. This KIP proposes to not expose > > controller metrics on KRaft brokers since KRaft brokers are not > controller > > eligible and will never have a non-zero value for the metric. Since > > exposing metrics that will always be zero is both unneeded and causes > > non-negligible performance impact it would be best to not move forward > with > > KAFKA-13140: https://github.com/apache/kafka/pull/11133 and instead > accept > > this KIP. > > > > > > > > Here is a link to the KIP which documents the behavior change from how > > controller metrics are exposed in a Kafka cluster using Zookeeper to how > > they are exposed in a Kafka cluster using KRaft. > > : > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP+771%3A+KRaft+brokers+should+not+expose+controller+metrics > > > > Here is a link to the discussion: > > > > > https://lists.apache.org/thread.html/r74432034527fab13cc973ad5187ef5881a642500d77b0d275dd7f018%40%3Cdev.kafka.apache.org%3E > > > > Regards, > > Ryan Dielhenn > > >