----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73300/#review222871 -----------------------------------------------------------
hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java Line 2932 (original), 2950 (patched) <https://reviews.apache.org/r/73300/#comment312042> Is this function needed any more? If not, can this be removed? hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java Line 2949 (original), 2967 (patched) <https://reviews.apache.org/r/73300/#comment312043> Please review and fix log message here and in line 2952 - Abhay Kulkarni On April 26, 2021, 8:25 p.m., Ramesh Mani wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73300/ > ----------------------------------------------------------- > > (Updated April 26, 2021, 8:25 p.m.) > > > Review request for ranger, Don Bosco Durai, Abhay Kulkarni, Madhan Neethiraj, > Mehul Parikh, Selvamohan Neethiraj, Sailaja Polavarapu, and Velmurugan > Periasamy. > > > Bugs: RANGER-3252 > https://issues.apache.org/jira/browse/RANGER-3252 > > > Repository: ranger > > > Description > ------- > > RANGER-3252:Inconsistent behavior in Ranger Role authorization within same > hive beeline session > > > Diffs > ----- > > > hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java > 5bd5c2da4 > > > Diff: https://reviews.apache.org/r/73300/diff/2/ > > > Testing > ------- > > - Within the Same Hive Session when Roles are updated for inclusion and > exclusion of user/group/roles, authorization failed for those users/ groups > and roles and this patch takes care of it > - verified by setting current role with "set role <rolename>" within the same > Hive session. > - verified show roles and show current roles based on inclusion and exclusion > of user / groups and roles. > > > Thanks, > > Ramesh Mani > >
