Hi Rajini, Sure, I will start a discussion thread soon on dev mailing list.
Thanks, Satish. On Thu, Aug 8, 2019 at 1:29 AM Rajini Sivaram <rajinisiva...@gmail.com> wrote: > > Hi Ron/Harsha/Satish, > > Thanks for reviewing the KIP! > > We should perhaps have a wider discussion outside this KIP for refactoring > clients so that others who are not following this KIP also notice the > discussion. Satish, would you like to start a discussion thread on dev? > > Regards, > > Rajini > > > On Wed, Aug 7, 2019 at 6:21 PM Satish Duggana <satish.dugg...@gmail.com> > wrote: > > > I felt the same need when we want to add a pluggable API for core > > server functionality. This does not need to be part of this KIP, it > > can be a separate KIP. I can contribute those refactoring changes if > > others are OK with that. > > > > It is better to have a structure like below. > > > > kafka-common: > > common classes which can be used in any of the other modules in Kafka > > like client, Kafka-server-common and server etc. > > > > kafka-client-common: > > common classes which can be used in the client module. This can be > > part of client module itself. > > > > kafka-server-common: > > classes required only for kafka-server. > > > > Thanks. > > Satish. > > > > On Wed, Aug 7, 2019 at 9:28 PM Harsha Chintalapani <ka...@harsha.io> > > wrote: > > > > > > Thanks for the KIP Rajini. > > > Quick thought, it would be good to have a common module outside of > > clients > > > that only applies to server side interfaces & changes. It looks like we > > are > > > increasingly in favor of using Java interface for pluggable modules on > > the > > > broker side. > > > > > > Thanks, > > > Harsha > > > > > > > > > On Tue, Aug 06, 2019 at 2:31 PM, Rajini Sivaram <rajinisiva...@gmail.com > > > > > > wrote: > > > > > > > Hi all, > > > > > > > > I have created a KIP to replace the Scala Authorizer API with a new > > Java > > > > API: > > > > > > > > - > > > > https://cwiki.apache.org/confluence/display/KAFKA/ > > > > KIP-504+-+Add+new+Java+Authorizer+Interface > > > > > > > > This is replacement for KIP-50 which was accepted but never merged. > > Apart > > > > from moving to a Java API consistent with other pluggable interfaces > > in the > > > > broker, KIP-504 also attempts to address known limitations in the > > > > authorizer. If you have come across other limitations that you would > > like > > > > to see addressed in the new API, please raise these on the discussion > > > > thread so that we can consider those too. All suggestions and feedback > > are > > > > welcome. > > > > > > > > Thank you, > > > > > > > > Rajini > > > > > >
