+1 Thanks Parth
On Wed, Mar 30, 2016 at 8:22 PM, Ewen Cheslack-Postava <e...@confluent.io> wrote: > +1 > > -Ewen > > On Wed, Mar 30, 2016 at 7:20 PM, Gwen Shapira <g...@confluent.io> wrote: > > > So my +1 is back :) > > > > On Wed, Mar 30, 2016 at 4:51 PM, Ashish Singh <asi...@cloudera.com> > wrote: > > > > > My bad, I moved the config option to rejected alternatives. The new > > method > > > getSupportedPrincipalTypes is what the KIP is proposing. > > > > > > > > > On Wed, Mar 30, 2016 at 4:43 PM, Gwen Shapira <g...@confluent.io> > wrote: > > > > > > > My appologies, but I need to retract my vote. > > > > > > > > The KIP currently specifies two alternatives (none rejected). So I'm > > not > > > > sure what I'm voting on... > > > > > > > > For reference, my original +1 was for the new method: > > > > getSupportedPrincipalTypes() > > > > I'm not even sure how would a config work in that case. > > > > > > > > Anyway, we need to resolve this before voting. > > > > > > > > Gwen > > > > > > > > On Wed, Mar 30, 2016 at 4:34 PM, Gwen Shapira <g...@confluent.io> > > wrote: > > > > > > > > > +1. > > > > > > > > > > It's a small change and I can see how this will help improve > > usability > > > > for > > > > > some of the authorizers. > > > > > > > > > > Gwen > > > > > > > > > > On Wed, Mar 30, 2016 at 11:02 AM, Ashish Singh < > asi...@cloudera.com> > > > > > wrote: > > > > > > > > > >> Hi Guys, > > > > >> > > > > >> I would like to open the vote for KIP-50. > > > > >> > > > > >> KIP: > > > > >> > > > > >> > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-50+-+Enhance+Authorizer+interface+to+be+aware+of+supported+Principal+Types > > > > >> > > > > >> Discuss thread: here > > > > >> < > > > > >> > > > > > > > > > > http://mail-archives.apache.org/mod_mbox/kafka-dev/201603.mbox/%3CCAGQG9cUCLDO0owdziDcL9iStXNF1wURyVNcEZedQJg%3DUuC7j%3DQ%40mail.gmail.com%3E > > > > >> > > > > > >> > > > > >> JIRA: https://issues.apache.org/jira/browse/KAFKA-3186 > > > > >> > > > > >> PR: https://github.com/apache/kafka/pull/861 > > > > >> > > > > >> -- > > > > >> > > > > >> Regards, > > > > >> Ashish > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > Regards, > > > Ashish > > > > > > > > > -- > Thanks, > Ewen >
