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

Review request for ranger, Anand Nadar, Abhay Kulkarni, Mehul Parikh, Monika 
Kachhadiya, Pradeep Agrawal, Ramesh Mani, and Subhrat Chaudhary.


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


Repository: ranger


Description
-------

- added following methods in GdsPolicyEngine:
-- getDatasetsForPrincipals(): get datasets for given principals
-- getProjectsForPrincipals(): get projects for given principals
-- getDatasetResources(): get resources in a dataset
-- getProjectResources(): get resources in a project
-- getDataShareResources(): get resources in a dataShare


Diffs
-----

  
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/gds/GdsDataShareEvaluator.java
 4936a86ed 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/gds/GdsDatasetEvaluator.java
 aa9fbea7c 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/gds/GdsDipEvaluator.java
 a5256ca4c 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/gds/GdsDshidEvaluator.java
 c32f22b27 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/gds/GdsPolicyEngine.java
 337134fe4 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/gds/GdsProjectEvaluator.java
 17a86163b 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/gds/GdsSharedResourceEvaluator.java
 726331271 
  
agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerPolicyEvaluator.java
 2856ccc6b 
  
agents-common/src/test/java/org/apache/ranger/plugin/policyengine/gds/TestGdsPolicyEngine.java
 9d1560562 
  agents-common/src/test/resources/policyengine/gds/gds_info_hive.json 
34fcc6290 
  
agents-common/src/test/resources/policyengine/gds/test_gds_policy_engine_hive.json
 2ab93bde6 


Diff: https://reviews.apache.org/r/74742/diff/1/


Testing
-------

- added unit tests to validate APIs
- verified that all existing tests pass successfully


Thanks,

Madhan Neethiraj

Reply via email to