----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73522/#review223354 -----------------------------------------------------------
security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java Lines 4847 (patched) <https://reviews.apache.org/r/73522/#comment312427> If there are no groups for given user when database query returns, will the groupNames be null (or empty set)? Please consider adding public group even when the groupNames is null ( by creating a HashSet containing only "public" group and assigniing it to groupNames). Then the test on line 4851 is not required. security-admin/src/main/java/org/apache/ranger/common/RangerConstants.java Lines 43 (patched) <https://reviews.apache.org/r/73522/#comment312426> This may not be necessary. It is already defined as RangerPolicyEngine.USER_CURRENT - Abhay Kulkarni On Aug. 18, 2021, 3:52 p.m., Mateen Mansoori wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73522/ > ----------------------------------------------------------- > > (Updated Aug. 18, 2021, 3:52 p.m.) > > > Review request for ranger, Dhaval Rajpara, Dhaval Shah, Abhay Kulkarni, > Madhan Neethiraj, Mehul Parikh, Pradeep Agrawal, and Velmurugan Periasamy. > > > Bugs: RANGER-3372 > https://issues.apache.org/jira/browse/RANGER-3372 > > > Repository: ranger > > > Description > ------- > > PROBLEM STATEMENT : > We have a user called "A" > User A belongs to 2 Groups [Group 1 and Group 2 ] > Group 1 is assigned to Hive Policy [Test 1 ] > Group 2 is assigned to Hive Policy [Test 2 ] > > Now while searching user "A" in Ranger Reports , We can only see 1 of the > Policy is listing , Expected behaviour should be : Both policy should be > displayed in reports. > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java > c5add3aa2 > security-admin/src/main/java/org/apache/ranger/common/RangerConstants.java > f00ea05ca > > > Diff: https://reviews.apache.org/r/73522/diff/2/ > > > Testing > ------- > > Ran - mvn clean compile test verify install > Testing - Tested on local VM. > > > Thanks, > > Mateen Mansoori > >