----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72947/#review222018 -----------------------------------------------------------
hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java Lines 2276 (patched) <https://reviews.apache.org/r/72947/#comment311090> Please enclose the debug statement inside if and update the logging format "className.MethodName". hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java Lines 2301 (patched) <https://reviews.apache.org/r/72947/#comment311091> Move this block at line 2309 - bhavik patel On Oct. 10, 2020, 9:53 a.m., ru jia wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72947/ > ----------------------------------------------------------- > > (Updated Oct. 10, 2020, 9:53 a.m.) > > > Review request for ranger. > > > Bugs: RANGER-3033 > https://issues.apache.org/jira/browse/RANGER-3033 > > > Repository: ranger > > > Description > ------- > > Currently, hive authorizer doesn't support command 'show role grant user > user1' > > > Diffs > ----- > > > hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java > 1bec50b37 > > > Diff: https://reviews.apache.org/r/72947/diff/1/ > > > Testing > ------- > > 1. create a role which contains user 'testuser' on Ranger web > 2. go into beeline(make sure kerberos login in security mode) > 3. execute 'show role grant user testuser', and then console will print the > detail out > > > Thanks, > > ru jia > >
