----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43977/#review122783 -----------------------------------------------------------
Ship it! - Hao Hao On March 9, 2016, 7:25 p.m., Ashish Singh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43977/ > ----------------------------------------------------------- > > (Updated March 9, 2016, 7:25 p.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 > >
