-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42287/
-----------------------------------------------------------
(Updated Jan. 29, 2016, 8:41 a.m.)
Review request for sentry.
Repository: sentry
Description
-------
Create CommonPrivilege for every component.
The main change is for the interface Privilege, implies(Privilege p) --->
implies(Privilege privilege, Map<String, String> implyMethodMap, Map<String,
Integer> actionMap)
The actionMap is stored the mapping between action name and action value, eg:
for Hive: select ---> 1 (binary: 0000001)
insert ---> 3 (binary: 0000011)
create ---> 4 (binary: 0000100)
all ---> 7 (binary: 0000111)
when compare the action, use the action value with the & operator
The implyMethodMap is stored the mapping between resource type and imply
method, eg:
for Hive: db ---> string (imply as string value)
url ---> url (imply as url value)
Diffs (updated)
-----
sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/ImplyMethodType.java
PRE-CREATION
sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/Model.java
PRE-CREATION
sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/Resource.java
PRE-CREATION
sentry-policy/sentry-policy-common/src/main/java/org/apache/sentry/policy/common/CommonPrivilege.java
PRE-CREATION
sentry-policy/sentry-policy-common/src/main/java/org/apache/sentry/policy/common/PolicyConstants.java
0bad8c1
sentry-policy/sentry-policy-common/src/test/java/org/apache/sentry/policy/common/ColumnResourceForTest.java
PRE-CREATION
sentry-policy/sentry-policy-common/src/test/java/org/apache/sentry/policy/common/DbResourceForTest.java
PRE-CREATION
sentry-policy/sentry-policy-common/src/test/java/org/apache/sentry/policy/common/ModelForTest.java
PRE-CREATION
sentry-policy/sentry-policy-common/src/test/java/org/apache/sentry/policy/common/ServerResourceForTest.java
PRE-CREATION
sentry-policy/sentry-policy-common/src/test/java/org/apache/sentry/policy/common/TableResourceForTest.java
PRE-CREATION
sentry-policy/sentry-policy-common/src/test/java/org/apache/sentry/policy/common/TestCommonPrivilege.java
PRE-CREATION
sentry-policy/sentry-policy-common/src/test/java/org/apache/sentry/policy/common/TestKeyValue.java
2dfc7c5
sentry-policy/sentry-policy-common/src/test/java/org/apache/sentry/policy/common/UrlResourceForTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/42287/diff/
Testing
-------
Thanks,
Colin Ma
