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