Hi, Just wanted to see if anyone had any concerns with this KIP. I would like to put this to vote soon, if there are no concerns.
Thanks, Mayuresh On Thu, Jan 12, 2017 at 11:21 AM, Mayuresh Gharat < gharatmayures...@gmail.com> wrote: > Hi Ismael, > > Fair point. I will update it. > > Thanks, > > Mayuresh > > On Thu, Jan 12, 2017 at 11:07 AM, Ismael Juma <ism...@juma.me.uk> wrote: > >> Hi Mayuresh, >> >> Thanks for the KIP. A quick comment before I do a more detailed analysis, >> the KIP says: >> >> `This KIP is a pure addition to existing functionality and does not >> include >> any backward incompatible changes.` >> >> However, the KIP is proposing the addition of a method to the >> PrincipalBuilder pluggable interface, which is not a compatible change. >> Existing implementations would no longer compile, for example. It would be >> good to make this clear in the KIP. >> >> Ismael >> >> On Thu, Jan 12, 2017 at 5:44 PM, Mayuresh Gharat < >> gharatmayures...@gmail.com >> > wrote: >> >> > Hi all. >> > >> > We created KIP-111 to propose that Kafka should preserve the Principal >> > generated by the PrincipalBuilder while processing the request received >> on >> > socket channel, on the broker. >> > >> > Please find the KIP wiki in the link >> > https://cwiki.apache.org/confluence/pages/viewpage.action? >> pageId=67638388. >> > We would love to hear your comments and suggestions. >> > >> > >> > Thanks, >> > >> > Mayuresh >> > >> > > > > -- > -Regards, > Mayuresh R. Gharat > (862) 250-7125 > -- -Regards, Mayuresh R. Gharat (862) 250-7125