-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42778/
-----------------------------------------------------------
(Updated Jan. 27, 2016, 6:16 p.m.)
Review request for sentry and Dapeng Sun.
Changes
-------
Address review comments.
Bugs: SENTRY-1013
https://issues.apache.org/jira/browse/SENTRY-1013
Repository: sentry
Description
-------
SENTRY-1013: Add policy engine for Kafka
Diffs (updated)
-----
pom.xml aa4a021eae93c3c817ec055e18868b95f7cba124
sentry-policy/pom.xml ef938a6b52bb00042ea8ae32da3d245ef0697a00
sentry-policy/sentry-policy-kafka/pom.xml PRE-CREATION
sentry-policy/sentry-policy-kafka/src/main/java/org/apache/sentry/policy/kafka/KafkaModelAuthorizables.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/main/java/org/apache/sentry/policy/kafka/KafkaPrivilegeValidator.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/main/java/org/apache/sentry/policy/kafka/KafkaWildcardPrivilege.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/main/java/org/apache/sentry/policy/kafka/SimpleKafkaPolicyEngine.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/test/java/org/apache/sentry/policy/kafka/KafkaPolicyFileProviderBackend.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/test/java/org/apache/sentry/policy/kafka/MockGroupMappingServiceProvider.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/test/java/org/apache/sentry/policy/kafka/TestKafkaModelAuthorizables.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/test/java/org/apache/sentry/policy/kafka/TestKafkaPrivilegeValidator.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/test/java/org/apache/sentry/policy/kafka/TestKafkaWildcardPrivilege.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/test/java/org/apache/sentry/policy/kafka/engine/AbstractTestKafkaPolicyEngine.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/test/java/org/apache/sentry/policy/kafka/engine/TestKafkaPolicyEngineDFS.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/test/java/org/apache/sentry/policy/kafka/engine/TestKafkaPolicyEngineLocalFS.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/test/java/org/apache/sentry/policy/kafka/provider/TestKafkaAuthorizationProviderGeneralCases.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/test/java/org/apache/sentry/policy/kafka/provider/TestKafkaAuthorizationProviderSpecialCases.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/test/java/org/apache/sentry/policy/kafka/provider/TestKafkaPolicyNegative.java
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/test/resources/log4j.properties
PRE-CREATION
sentry-policy/sentry-policy-kafka/src/test/resources/test-authz-provider.ini
PRE-CREATION
Diff: https://reviews.apache.org/r/42778/diff/
Testing
-------
Tested via unit tests and end-to-end tests that will be added as part of
SENTRY-1014.
Thanks,
Ashish Singh
