1. It is intended to return broker metrics. 2. I did searching, there is no any public API return broker Metrics class so far. So this will be first time. I did find connect also expose the Metrics class, but it is not in public API. org.apache.kafka.connect.runtime.ConnectMetrics /** * Get the {@link Metrics Kafka Metrics} that are managed by this object and that should be used to * add sensors and individual metrics. * * @return the Kafka Metrics instance; never null */ public Metrics metrics() { return metrics; }
On 2020/05/06 15:31:07, Ismael Juma <ism...@juma.me.uk> wrote: > Thanks for the KIP. A couple of questions: > > 1. Is it intended for this method to return null or the broker metrics > instance? > 2. Is the Metrics class returned in any public APIs today or this the first > time we are doing it? > > Ismael > > On Wed, May 6, 2020 at 8:10 AM Rajini Sivaram <rajinisiva...@gmail.com> > wrote: > > > Hi Jeff, > > > > Thanks for the KIP. It looks useful since it allows authorizers to use the > > broker's metrics instance. We could perhaps use this in AclAuthorizer to > > generate authorizer metrics? > > > > Regards, > > > > Rajini > > > > On Tue, May 5, 2020 at 9:04 PM Zhiguo Huang <jeff.hu...@confluent.io> > > wrote: > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-608+-+Add+a+new+method+to+AuthorizerServerInfo+Interface > > > > > >