Hey, Thanks for the KIP.
I think that having the possibilities to expose metrics in plugins such as the authorizer in a nice improvement. I do wonder if we could come up with a more generic way to do this that would apply to all plugins instead of having something specific for the authorized. For instance, we have the `Configurable` interface that allows us to configure a plugin. We could perhaps think of having a `Monitorable` interface that would allow to pass the Metrics instance to the plugin. Have you thought about such an approach? Best, David On Wed, May 6, 2020 at 6:43 PM Jeff Huang <jeff.hu...@confluent.io> wrote: > Will update KIP. > thanks! > > On 2020/05/06 15:09:53, 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 > > > > > >
