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




plugin-atlas/src/main/java/org/apache/ranger/services/atlas/RangerServiceAtlas.java
Line 164 (original), 173 (patched)
<https://reviews.apache.org/r/71155/#comment304107>

    Add PUBLIC_GROUPNAME_DEFAULT group to this policy-item, to allow everyone 
to read entities - which is the ask in RANGER-2516.



plugin-atlas/src/main/java/org/apache/ranger/services/atlas/RangerServiceAtlas.java
Lines 219 (patched)
<https://reviews.apache.org/r/71155/#comment304106>

    For better readability of the created Ranger policy, consider granting to 
user RangerPolicyEngine.USER_CURRENT instead of  PUBLIC_GROUPNAME_DEFAULT group.
    
      RangerPolicyItem item = new RangerPolicyItem(accesses, 
Arrays.asList(RangerPolicyEngine.USER_CURRENT), null, null, null, false);


- Madhan Neethiraj


On July 25, 2019, 12:38 p.m., Nikhil P wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/71155/
> -----------------------------------------------------------
> 
> (Updated July 25, 2019, 12:38 p.m.)
> 
> 
> Review request for ranger, Gautam Borad, Abhay Kulkarni, Madhan Neethiraj, 
> Nixon Rodrigues, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja 
> Polavarapu, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-2516
>     https://issues.apache.org/jira/browse/RANGER-2516
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Update Ranger default policies to provide entity-read access to public group
> 
> 
> Diffs
> -----
> 
>   
> plugin-atlas/src/main/java/org/apache/ranger/services/atlas/RangerServiceAtlas.java
>  459e791c3 
> 
> 
> Diff: https://reviews.apache.org/r/71155/diff/2/
> 
> 
> Testing
> -------
> 
> Tested on local if additional policy is getting created for required 
> resources.
> 
> 
> Thanks,
> 
> Nikhil P
> 
>

Reply via email to