----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45600/#review126929 -----------------------------------------------------------
Ship it! Ship It! - Madhan Neethiraj On April 4, 2016, 7:54 p.m., Abhay Kulkarni wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45600/ > ----------------------------------------------------------- > > (Updated April 4, 2016, 7:54 p.m.) > > > Review request for ranger and Madhan Neethiraj. > > > Bugs: RANGER-903 > https://issues.apache.org/jira/browse/RANGER-903 > > > Repository: ranger > > > Description > ------- > > Optimized processing of ServiceTags to minimize creation/deletion of > Tag-related objects > > > Diffs > ----- > > agents-common/src/main/java/org/apache/ranger/plugin/model/RangerTag.java > 78040ba > > agents-common/src/main/java/org/apache/ranger/plugin/store/AbstractTagStore.java > 7d2e130 > agents-common/src/main/java/org/apache/ranger/plugin/store/TagStore.java > 69bd628 > > agents-common/src/main/java/org/apache/ranger/plugin/store/file/TagFileStore.java > 58052cf > agents-common/src/main/java/org/apache/ranger/plugin/util/ServiceTags.java > 738a947 > > agents-common/src/test/java/org/apache/ranger/plugin/store/TestTagStore.java > aaace89 > security-admin/db/sqlserver/patches/021-update-tag-for-owner.sql dabe841 > > security-admin/src/main/java/org/apache/ranger/biz/RangerTagDBRetriever.java > 3b7555e > security-admin/src/main/java/org/apache/ranger/biz/TagDBStore.java cc2386e > > security-admin/src/main/java/org/apache/ranger/common/RangerServiceTagsCache.java > cdc44e0 > security-admin/src/main/java/org/apache/ranger/db/XXServiceResourceDao.java > c990ae7 > security-admin/src/main/java/org/apache/ranger/db/XXTagAttributeDao.java > c993477 > security-admin/src/main/java/org/apache/ranger/db/XXTagDao.java 3c9370b > security-admin/src/main/java/org/apache/ranger/entity/XXTag.java 526557e > security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java > 0dbd042 > > security-admin/src/main/java/org/apache/ranger/rest/ServiceTagsProcessor.java > ebe71eb > security-admin/src/main/java/org/apache/ranger/rest/TagREST.java 378ff0b > > security-admin/src/main/java/org/apache/ranger/service/RangerServiceResourceService.java > 5cf26c9 > security-admin/src/main/resources/META-INF/jpa_named_queries.xml 2bb66ca > tagsync/samples/tags.json 3028f9d > > tagsync/src/main/java/org/apache/ranger/tagsync/source/atlas/AtlasNotificationMapper.java > 2168983 > tagsync/src/main/resources/etc/ranger/data/tags.json b4cd736 > > Diff: https://reviews.apache.org/r/45600/diff/ > > > Testing > ------- > > Uploaded ServiceTags containing Tag objects which already existed in the > Ranger. Ensured that existing objects where updated wherever necessary, but > no unnecessary deletions/creations were done. > > > Thanks, > > Abhay Kulkarni > >
