----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35162/#review86881 -----------------------------------------------------------
Ship it! hbase-agent/src/main/java/org/apache/ranger/authorization/hbase/RangerAuthorizationCoprocessor.java <https://reviews.apache.org/r/35162/#comment139022> should access be reset here as well? hbase-agent/src/main/java/org/apache/ranger/authorization/hbase/RangerAuthorizationFilter.java <https://reviews.apache.org/r/35162/#comment139023> make sure that access is set correctly by the time this code is executed. - Madhan Neethiraj On June 5, 2015, 11:28 p.m., Alok Lal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35162/ > ----------------------------------------------------------- > > (Updated June 5, 2015, 11:28 p.m.) > > > Review request for ranger and Madhan Neethiraj. > > > Bugs: RANGER-533 > https://issues.apache.org/jira/browse/RANGER-533 > > > Repository: ranger > > > Description > ------- > > - Changed code for filter and authorizer. > - Added misc logging to some classes for ease of debuggin > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerPolicyValidator.java > cea3e05 > > hbase-agent/src/main/java/org/apache/ranger/authorization/hbase/AuthorizationSession.java > e0b652e > > hbase-agent/src/main/java/org/apache/ranger/authorization/hbase/HbaseAuditHandlerImpl.java > e383614 > > hbase-agent/src/main/java/org/apache/ranger/authorization/hbase/RangerAuthorizationCoprocessor.java > abf8a33 > > hbase-agent/src/main/java/org/apache/ranger/authorization/hbase/RangerAuthorizationFilter.java > ae61a1e > > hbase-agent/src/test/java/org/apache/ranger/authorization/hbase/RangerAuthorizationFilterTest.java > 4b49721 > > Diff: https://reviews.apache.org/r/35162/diff/ > > > Testing > ------- > > Testing scenario laid out in the apache JIRA. > > > Thanks, > > Alok Lal > >
