> On Jan. 26, 2016, 6:16 a.m., Dapeng Sun wrote: > > Thank Ashish, Here are my comments.
Thanks for the review Dapeng. > On Jan. 26, 2016, 6:16 a.m., Dapeng Sun wrote: > > sentry-policy/sentry-policy-kafka/src/main/java/org/apache/sentry/policy/kafka/KafkaWildcardPrivilege.java, > > line 62 > > <https://reviews.apache.org/r/42778/diff/1/?file=1221122#file1221122line62> > > > > We need to separate this part logic from Kafka, Sqoop and Solr... and > > add them to a common abstract privilege. I think we should file a jira at > > Sentry to refine it. Created https://issues.apache.org/jira/browse/SENTRY-1028 - Ashish ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42778/#review116273 ----------------------------------------------------------- On Jan. 26, 2016, 4:40 a.m., Ashish Singh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42778/ > ----------------------------------------------------------- > > (Updated Jan. 26, 2016, 4:40 a.m.) > > > Review request for sentry and Dapeng Sun. > > > Bugs: SENTRY-1013 > https://issues.apache.org/jira/browse/SENTRY-1013 > > > Repository: sentry > > > Description > ------- > > SENTRY-1013: Add policy engine for Kafka > > > Diffs > ----- > > 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/KafkaWildcardPrivilege.java > PRE-CREATION > > sentry-policy/sentry-policy-kafka/src/main/java/org/apache/sentry/policy/kafka/ResourceRequiredMatch.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/TestKafkaWildcardPrivilege.java > PRE-CREATION > > sentry-policy/sentry-policy-kafka/src/test/java/org/apache/sentry/policy/kafka/TestResourceNameRequiredMatch.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 > >
