Hey Colin and David,

I added another table for the ZK version of RegisteredBrokerCount.

Best,
Ryan Dielhenn

On 2021/06/04 08:21:27, David Jacot <dja...@confluent.io.INVALID> wrote: 
> Hi Ryan,
> 
> Thanks for the KIP.
> 
> +1 for adding RegisteredBrokerCount to the ZK controller as well. This
> would be really helpful.
> 
> Best,
> David
> 
> On Fri, Jun 4, 2021 at 12:44 AM Colin McCabe <cmcc...@apache.org> wrote:
> 
> > Hi Ryan,
> >
> > Thanks for the KIP. I think it would be good to provide the
> > RegisteredBrokerCount metric for the ZK controller as well as for the
> > Quorum controller. Looks good aside from that!
> >
> > best,
> > Colin
> >
> > On Thu, Jun 3, 2021, at 14:09, Ryan Dielhenn wrote:
> > > Hey kafka-dev,
> > >
> > > I created KIP-748 as a proposal to add broker count metrics to the
> > > Quorum Controller.
> > >
> > >
> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-748%3A+Add+Broker+Count+Metrics#KIP748:AddBrokerCountMetrics
> > >
> > > Best,
> > > Ryan Dielhenn
> > >
> >
> 

Reply via email to