----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72299/ -----------------------------------------------------------
(Updated April 3, 2020, 6:12 a.m.) Review request for ranger, Ankita Sinha, Gautam Borad, Abhay Kulkarni, Mehul Parikh, Pradeep Agrawal, Vishal Suvagia, and Velmurugan Periasamy. Changes ------- added ranger community for review Repository: ranger Description ------- Adding the functionality of merging policy while creation of the policy. The following conditions should match in order to merge the policy. 1. There should be already existing policy. 2. One should pass param mergeIfExists=true in create Policy API 3. One should also pass param "serviceName" and "policyName". 4. You can pass "zoneName" if available. Diffs ----- security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 8618f3279 Diff: https://reviews.apache.org/r/72299/diff/1/ Testing ------- 1. Existing create or update policy. 2. Creating and updating policy using postman 3. Creating policy if already exist using below param a) mergeIfExists=true b) serviceName c) policyName expected behavior policy should get merged in existing policy. 4. Creating policy if already exist using below param a) updateIfExists=true b) serviceName c) policyName expected behavior policy should get updated with existing policy. 5. Creating policy if already exist using below param a) deleteIfExists=true b) serviceName c) policyName expected behavior existing policy should get deleted and new policy is created. Thanks, Dineshkumar Yadav
