----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72325/ -----------------------------------------------------------
(Updated April 13, 2020, 12:51 p.m.) Review request for ranger, Ankita Sinha, Gautam Borad, Kishor Gollapalliwar, Abhay Kulkarni, Mehul Parikh, Pradeep Agrawal, Vishal Suvagia, and Velmurugan Periasamy. Changes ------- Removing policy condition functionality when policy send with mergeIfExist=true Repository: ranger Description ------- Handling the scenario when non-existing policy send with mergeIfExist=true Diffs (updated) ----- security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 38b49820b security-admin/src/main/java/org/apache/ranger/rest/ServiceRESTUtil.java d85028c95 Diff: https://reviews.apache.org/r/72325/diff/4/ Changes: https://reviews.apache.org/r/72325/diff/3-4/ Testing (updated) ------- Test case: Creating non-existing policy with below param a) mergeIfExists=true b) serviceName c) policyName Expected: New policy should be created. Creating policy with below param and role which is already exist a) mergeIfExists=true b) serviceName c) policyName Expected: role should get merged. Thanks, Dineshkumar Yadav
