ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dineshkumar Yadav <dineshkumar.ya...@outlook.com>
Subject Re: Review Request 72325: RANGER-2772 : Adding the functionality of merging the policy
Date Wed, 08 Apr 2020 10:18:33 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated April 8, 2020, 10:18 a.m.)

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


Included role merge at the time of policy merge

Repository: ranger


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/2/

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

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.


Dineshkumar Yadav

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message