-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37884/
-----------------------------------------------------------
(Updated Aug. 28, 2015, 6:12 p.m.)
Review request for ranger, Gautam Borad and Abhay Kulkarni.
Changes
-------
RANGER-595: replaced references to externalId with guid, to be consistent with
tag-store
Bugs: RANGER-595
https://issues.apache.org/jira/browse/RANGER-595
Repository: ranger
Description
-------
Updated tag-store implementation with fixes in DB persistence and new methods
Diffs (updated)
-----
agents-common/src/main/java/org/apache/ranger/plugin/model/RangerServiceResource.java
3728f6d
agents-common/src/main/java/org/apache/ranger/plugin/model/RangerTag.java
25dc24d
agents-common/src/main/java/org/apache/ranger/plugin/store/TagPredicateUtil.java
950969e
agents-common/src/main/java/org/apache/ranger/plugin/store/TagValidator.java
ada5dd2
agents-common/src/main/java/org/apache/ranger/plugin/store/file/TagFileStore.java
8a63f72
agents-common/src/main/java/org/apache/ranger/plugin/util/SearchFilter.java
9dc4994
agents-common/src/test/java/org/apache/ranger/plugin/store/TestTagStore.java
797b083
security-admin/db/mysql/patches/016-updated-schema-for-tag-based-policy.sql
4f6fe02
security-admin/src/main/java/org/apache/ranger/biz/TagDBStore.java f50687a
security-admin/src/main/java/org/apache/ranger/entity/XXTag.java 46b760b
security-admin/src/main/java/org/apache/ranger/rest/TagREST.java b667777
security-admin/src/main/java/org/apache/ranger/rest/TagRESTConstants.java
915ff13
security-admin/src/main/java/org/apache/ranger/service/RangerServiceResourceService.java
PRE-CREATION
security-admin/src/main/java/org/apache/ranger/service/RangerTagDefService.java
bd61ff1
security-admin/src/main/java/org/apache/ranger/service/RangerTagServiceBase.java
7a9c9da
security-admin/src/main/resources/META-INF/jpa_named_queries.xml 555dfb0
Diff: https://reviews.apache.org/r/37884/diff/
Testing
-------
Verified that unit tests continued to succeed after this update. Also verified
DB store implementation using TestREST API.
Thanks,
Madhan Neethiraj
