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