-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72957/
-----------------------------------------------------------

(Updated Oct. 14, 2020, 9:14 a.m.)


Review request for ranger, Ankita Sinha, Gautam Borad, Kishor Gollapalliwar, 
Abhay Kulkarni, Mehul Parikh, Pradeep Agrawal, and Velmurugan Periasamy.


Changes
-------

code changes : considered policy name for selecting from multiple policy with 
same resource-signature


Repository: ranger


Description (updated)
-------

If updateifexist/mergeIfExist parameter passed while importing the ranger 
policies then additional policies are created for disabled policy with same 
resource.

Solution: 1. New disabled policy will be created only if no resource-signature 
matching policy found in ranger.
          2. If policy with same resource-signature found in Ranger then policy 
get merged or updated according to parameter passed during import.
          3. If multiple disabled policy with same resource-signature found 
then policy with same name will be picked up for update/merge otherwise new 
policy will be created.


Diffs (updated)
-----

  security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 
4ef9abeb8 


Diff: https://reviews.apache.org/r/72957/diff/2/

Changes: https://reviews.apache.org/r/72957/diff/1-2/


Testing
-------

Tested for below scenario 
    1. With no disable policy 
    2. With single disable policy 
    3. With multiple disable policy


Thanks,

Dineshkumar Yadav

Reply via email to