----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43977/#review122854 -----------------------------------------------------------
sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/TestAuthorize.java (line 52) <https://reviews.apache.org/r/43977/#comment184945> Thank Ashish for your contribution, I didn't find AbstractKafkaSentryTestBase, is it missing? https://github.com/apache/incubator-sentry/tree/kafka/sentry-tests/sentry-tests-kafka/src/main/java/org/apache/sentry/tests/e2e/kafka Thank Ashish for your contribution, I have only one comment. - Dapeng Sun On 三月 10, 2016, 3:25 a.m., Ashish Singh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43977/ > ----------------------------------------------------------- > > (Updated 三月 10, 2016, 3:25 a.m.) > > > Review request for sentry, Anne Yu, Dapeng Sun, and Hao Hao. > > > Bugs: SENTRY-1014 > https://issues.apache.org/jira/browse/SENTRY-1014 > > > Repository: sentry > > > Description > ------- > > SENTRY-1014: Add E2E tests for Kafka plugin. > > > Diffs > ----- > > > sentry-tests/sentry-tests-kafka/src/main/java/org/apache/sentry/tests/e2e/kafka/KafkaTestServer.java > 129191ae6f9fa4cc6794117fddb693a3cd1ab815 > > sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/TestAuthorize.java > PRE-CREATION > sentry-tests/sentry-tests-kafka/src/test/resources/user1.keystore.jks > PRE-CREATION > sentry-tests/sentry-tests-kafka/src/test/resources/user1.truststore.jks > PRE-CREATION > sentry-tests/sentry-tests-kafka/src/test/resources/user2.keystore.jks > PRE-CREATION > sentry-tests/sentry-tests-kafka/src/test/resources/user2.truststore.jks > PRE-CREATION > > Diff: https://reviews.apache.org/r/43977/diff/ > > > Testing > ------- > > Added E2E tests. > > Note that this contains changes from SENTRY-1098, SENTRY-1056, SENTRY-1030 > and SENTRY-1057, and will have to be rebased once they get it. > > > Thanks, > > Ashish Singh > >
