----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41374/#review110349 -----------------------------------------------------------
Ship it! Ship It! - Madhan Neethiraj On Dec. 15, 2015, 12:22 a.m., Alok Lal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41374/ > ----------------------------------------------------------- > > (Updated Dec. 15, 2015, 12:22 a.m.) > > > Review request for ranger, Don Bosco Durai and Madhan Neethiraj. > > > Bugs: RANGER-777 > https://issues.apache.org/jira/browse/RANGER-777 > > > Repository: ranger > > > Description > ------- > > Since kafka's support for authorizer was not on master, building kafka plugin > required some doing. Kafka team has since released the authorization changes > as part of the v0.9 jar that was pushed into maven central. As part of that > a few of the auxiliary APIs, related to authentication, that kafka plugin was > relying on got changed. Accordingly, this patch: > - Changes kafka plugin code so it uses the APIs that are released as part of > v0.9 kafka jar. > - Removes the need for special profile to build kafka plugin such that now it > gets build by default if a complete build is done for kafka > > These changes would be ported also port to 0.5. > > > Diffs > ----- > > > plugin-kafka/src/main/java/org/apache/ranger/authorization/kafka/authorizer/RangerKafkaAuthorizer.java > 2adf5d5 > pom.xml d3a7a99 > > Diff: https://reviews.apache.org/r/41374/diff/ > > > Testing > ------- > > 1. Kafka authorization for publish/consume works as before. > 2. Authentication API changes were used by HDFS audit plugin. Ensured that > audit to HDFS works before and after use of new API. Test was done against > kafka cluster. > > > Thanks, > > Alok Lal > >
