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 >> > >
