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/28 11:20:22 UTC

Review Request 38801: RANGER-660: updates for the review comments in TagSync implementation

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

Review request for ranger and Abhay Kulkarni.


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


Repository: ranger


Description
-------

Update for the review comments in https://reviews.apache.org/r/38652/


Diffs
-----

  agents-common/src/main/java/org/apache/ranger/plugin/model/RangerTagResourceMap.java 3b69ee7 
  agents-common/src/main/java/org/apache/ranger/plugin/store/TagStore.java 1c48ea0 
  agents-common/src/main/java/org/apache/ranger/plugin/store/TagValidator.java 4eda166 
  agents-common/src/main/java/org/apache/ranger/plugin/store/file/TagFileStore.java 27b4779 
  agents-common/src/main/java/org/apache/ranger/plugin/util/ServiceTags.java e56e942 
  pom.xml aa40882 
  security-admin/src/main/java/org/apache/ranger/biz/TagDBStore.java 962bfde 
  security-admin/src/main/java/org/apache/ranger/db/XXTagResourceMapDao.java 0cac1ef 
  security-admin/src/main/java/org/apache/ranger/rest/ServiceTagsProcessor.java PRE-CREATION 
  security-admin/src/main/java/org/apache/ranger/rest/TagREST.java dfa4a64 
  security-admin/src/main/java/org/apache/ranger/rest/TagRESTConstants.java 977e487 
  security-admin/src/main/java/org/apache/ranger/service/RangerTagResourceMapService.java d50a937 
  security-admin/src/main/resources/META-INF/jpa_named_queries.xml a5f4a6c 
  src/main/assembly/tagsync.xml d7262ac 
  tagsync/conf.dist/log4j.xml fb6986f 
  tagsync/conf.dist/ranger-tagsync-default-site.xml b098740 
  tagsync/conf.dist/ranger-tagsync-site.xml PRE-CREATION 
  tagsync/pom.xml 83011d2 
  tagsync/scripts/ranger-tagsync-services.sh 5ca5b95 
  tagsync/src/main/java/org/apache/ranger/model/TagSink.java 74ea5cf 
  tagsync/src/main/java/org/apache/ranger/model/TagSource.java 568d3c7 
  tagsync/src/main/java/org/apache/ranger/process/TagSyncConfig.java 1c995b1 
  tagsync/src/main/java/org/apache/ranger/process/TagSynchronizer.java c7e8d6e 
  tagsync/src/main/java/org/apache/ranger/sink/policymgr/TagRESTSink.java 98410d6 
  tagsync/src/main/java/org/apache/ranger/source/atlas/AtlasNotificationMapper.java PRE-CREATION 
  tagsync/src/main/java/org/apache/ranger/source/atlas/TagAtlasSource.java PRE-CREATION 
  tagsync/src/main/java/org/apache/ranger/source/file/TagFileSource.java 2952edb 
  tagsync/src/main/resources/ranger-tagsync-default.xml PRE-CREATION 
  tagsync/src/main/resources/ranger-tagsync-site.xml 63b9727 
  tagsync/src/test/java/org/apache/ranger/process/TestTagSynchronizer.java d810048 

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


Testing
-------


Thanks,

Madhan Neethiraj