You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Madhan Neethiraj <ma...@apache.org> on 2015/09/26 09:48:16 UTC

Review Request 38784: RANGER-274: removed unused config and conditions from tag servicedef; fixed tag-lookup implementation

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

Review request for ranger, Gautam Borad and Abhay Kulkarni.


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


Repository: ranger


Description
-------

RANGER-274: removed unused config and conditions from tag servicedef; fixed tag-lookup implementation


Diffs
-----

  agents-common/src/main/java/org/apache/ranger/plugin/store/TagStore.java e27947f 
  agents-common/src/main/java/org/apache/ranger/plugin/store/file/TagFileStore.java dd8fd5c 
  agents-common/src/main/java/org/apache/ranger/services/tag/RangerServiceTag.java 375f0b8 
  agents-common/src/main/resources/service-defs/ranger-servicedef-tag.json dfadd47 
  security-admin/src/main/java/org/apache/ranger/biz/ServiceMgr.java 641ab91 
  security-admin/src/main/java/org/apache/ranger/biz/TagDBStore.java f2a1ad9 
  security-admin/src/main/java/org/apache/ranger/db/XXTagDefDao.java 97b53fd 
  security-admin/src/main/java/org/apache/ranger/rest/TagREST.java 520c1be 
  security-admin/src/main/resources/META-INF/jpa_named_queries.xml 375e977 
  tagsync/src/main/java/org/apache/ranger/sink/policymgr/TagRESTSink.java 0695cd3 

Diff: https://reviews.apache.org/r/38784/diff/


Testing
-------

Verified that changes to servicedef and the fix to lookup work as expected


Thanks,

Madhan Neethiraj