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

Review request for ranger, Ankita Sinha, Asit Vadhavkar, Abhay Kulkarni, Mehul 
Parikh, Monika Kachhadiya, Mugdha Varadkar, Pradeep Agrawal, Prashant Satam, 
Ramesh Mani, and Subhrat Chaudhary.


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


Repository: ranger


Description
-------

- updated following REST APIs to support retrieval of datasets/projects that 
are shared with the caller (query param: sharedWithMe)
-- service/gds/dataset
-- service/gds/dataset/summary
-- service/gds/project
-- service/gds/project/summary
- introduced GdsPolicyAdminCache to cache GDS policies
- added GdsPolicyEngine API to retrive resources in datasets/dataShares/projecs
- renamed GdsPolicyEngine.getDatasetsForPrincipals() => 
GdsPolicyEngine.getDatasetsSharedWith()
- renamed GdsPolicyEngine.getProjectsForPrincipals() => 
GdsPolicyEngine.getProjectsSharedWith()


Diffs
-----

  
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/gds/GdsPolicyEngine.java
 541f0754e 
  agents-common/src/main/java/org/apache/ranger/plugin/util/SearchFilter.java 
548963838 
  
agents-common/src/test/java/org/apache/ranger/plugin/policyengine/gds/TestGdsPolicyEngine.java
 fb6d593e9 
  
agents-common/src/test/resources/policyengine/gds/test_gds_policy_engine_hive.json
 bcdf5a342 
  security-admin/src/main/java/org/apache/ranger/biz/GdsDBStore.java 292caf981 
  security-admin/src/main/java/org/apache/ranger/biz/GdsPolicyAdminCache.java 
PRE-CREATION 
  security-admin/src/main/java/org/apache/ranger/common/RangerSearchUtil.java 
8c9a6c13d 
  
security-admin/src/main/java/org/apache/ranger/common/ServiceGdsInfoCache.java 
b27a32265 
  security-admin/src/main/java/org/apache/ranger/util/RangerAdminCache.java 
PRE-CREATION 
  
security-admin/src/main/java/org/apache/ranger/util/RangerCacheDBValueLoader.java
 7b5da29c3 


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


Testing
-------

- updated tests to cover new APIs in GdsPolicyEngine
- verified that "GET /service/gds/dataset?sharedWithMe=true" returns only the 
datasets shared with the caller


Thanks,

Madhan Neethiraj

Reply via email to