----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46768/#review130915 -----------------------------------------------------------
Colm - can you please confirm that the HDFS plugin generated with this update works without running into any issues? If you verified this, the patch is good to go. Thanks. - Madhan Neethiraj On April 28, 2016, 10:48 a.m., Colm O hEigeartaigh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46768/ > ----------------------------------------------------------- > > (Updated April 28, 2016, 10:48 a.m.) > > > Review request for ranger. > > > Repository: ranger > > > Description > ------- > > Some minor changes following a review of the HDFS code: > > a) Remove unnecessary dependencies in both the HDFS shim + plugin > b) Some trivial changes to the RangerHdfsAuthorizer - use an empty Set for > FS.NONE instead of creating a new Set + use an existing constructor for the > RangerHdfsAccessRequest. > > > Diffs > ----- > > hdfs-agent/pom.xml f5e42eb > > hdfs-agent/src/main/java/org/apache/ranger/authorization/hadoop/RangerHdfsAuthorizer.java > 93dca87 > ranger-hdfs-plugin-shim/pom.xml 0522697 > > Diff: https://reviews.apache.org/r/46768/diff/ > > > Testing > ------- > > > Thanks, > > Colm O hEigeartaigh > >
