----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62437/#review185812 -----------------------------------------------------------
security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java Line 2929 (original), 2930 (patched) <https://reviews.apache.org/r/62437/#comment262157> Please conside doing Set<String> uniqueValues = new LinkedHashSet<>(values); and remove the code for finding the uniqueValues. You can trim the output later when you sent it if needed. - Ramesh Mani On Sept. 20, 2017, 12:53 p.m., Nikhil P wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62437/ > ----------------------------------------------------------- > > (Updated Sept. 20, 2017, 12:53 p.m.) > > > Review request for ranger, Ankita Sinha, Don Bosco Durai, Gautam Borad, Abhay > Kulkarni, Madhan Neethiraj, Pradeep Agrawal, Ramesh Mani, Selvamohan > Neethiraj, Sailaja Polavarapu, and Velmurugan Periasamy. > > > Bugs: RANGER-1779 > https://issues.apache.org/jira/browse/RANGER-1779 > > > Repository: ranger > > > Description > ------- > > 1) create a policy with multiple resource *,default using public api > 2) go to ranger admin ui and update the policy without any change > 3) again view the policy. > Issue: > default gets duplicated as resource in the policy. > and even new entry is added in resource map table for the last resource. > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java > 63fdf4f > > security-admin/src/main/java/org/apache/ranger/patch/PatchForNifiResourceUpdateExclude_J10008.java > 634082c > > > Diff: https://reviews.apache.org/r/62437/diff/1/ > > > Testing > ------- > > 1)Verified if same resource does not get duplicated during create and update > policy. > 2)Verified if resource duplication does not happen through public API and > Public APIv2 as well. > 3)Verified if policies are getting created with multiple distinct resource. > > > Thanks, > > Nikhil P > >
