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

(Updated Aug. 24, 2019, 1:30 a.m.)


Review request for ranger, Madhan Neethiraj and Ramesh Mani.


Changes
-------

Addressed review comment


Summary (updated)
-----------------

RANGER-2548: Ranger-admin updates to ensure owner information in 
GrantRevokeData is correctly consumed


Bugs: RANGER-2548
    https://issues.apache.org/jira/browse/RANGER-2548


Repository: ranger


Description
-------

Grant/Revoke requests are processed by Ranger to create/update corresponding 
Ranger Policy. With Hive support for OWNER, Ranger also needs to ensure that 
Grant/Revoke requests, if they provide ownership information for the Hive 
object being granted permission on, are processed correctly in accordance with 
prevailing Ranger policies.


Diffs (updated)
-----

  
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngineImpl.java
 daa62f408 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerTagAccessRequest.java
 cf590f9aa 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java
 ecd6cb746 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyItemEvaluator.java
 5bbbecedb 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerOptimizedPolicyEvaluator.java
 00c0d42ed 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerPolicyItemEvaluator.java
 ec3950f62 
  
agents-common/src/main/java/org/apache/ranger/plugin/service/RangerAuthContext.java
 02f343196 
  
agents-common/src/main/java/org/apache/ranger/plugin/util/RangerAccessRequestUtil.java
 c8276f127 
  security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 
bb825b822 


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

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


Testing (updated)
-------

Passed all unit tests


Thanks,

Abhay Kulkarni

Reply via email to