----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63178/#review188842 -----------------------------------------------------------
Ship it! Ship It! - Sergio Pena On Oct. 20, 2017, 12:41 p.m., kalyan kumar kalvagadda wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63178/ > ----------------------------------------------------------- > > (Updated Oct. 20, 2017, 12:41 p.m.) > > > Review request for sentry, Na Li and Sergio Pena. > > > Bugs: SENTRY-1989 > https://issues.apache.org/jira/browse/SENTRY-1989 > > > Repository: sentry > > > Description > ------- > > Bump Kafka version to the latest version available. 0.9 was released last > year (2016) and 0.11.1 is new in 2017 > > Here is the summary of code changes > 1. Fixed the API incomatability > 2. updated pom files to user updated version of kafka > 3. Added new property to kafka server. > 3. Refatored the the tests and commented one of the tests becausae of the > known issue with the latest version of kafka. > > > Diffs > ----- > > pom.xml 1c05556 > sentry-tests/sentry-tests-kafka/pom.xml c65323d > > sentry-tests/sentry-tests-kafka/src/main/java/org/apache/sentry/tests/e2e/kafka/KafkaTestServer.java > faeb369 > > sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/TestAclsCrud.java > a52c8d6 > > sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/TestAuthorize.java > 6d2cabf > > > Diff: https://reviews.apache.org/r/63178/diff/1/ > > > Testing > ------- > > Unit tests passed. > > > Thanks, > > kalyan kumar kalvagadda > >
