----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41374/#review116245 -----------------------------------------------------------
Ship it! Ship It! - Madhan Neethiraj On Jan. 26, 2016, 12:50 a.m., Alok Lal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41374/ > ----------------------------------------------------------- > > (Updated Jan. 26, 2016, 12:50 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 > ----- > > pom.xml 23ef49e > > 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 > >
