----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63308/ -----------------------------------------------------------
(Updated Oct. 26, 2017, 5:26 p.m.) Review request for sentry, Colm O hEigeartaigh and Sergio Pena. Bugs: SENTRY-2020 https://issues.apache.org/jira/browse/SENTRY-2020 Repository: sentry Description ------- with the new Kafka version, multiple threads will be requesting sentry for authorization at the same time for a same authorizable so the it is not guarentted to get the same exception when multiple permissions are missing. Code change is added to make sure that test suceeds if any one of the expected exception is thrown. Diffs (updated) ----- sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/AbstractKafkaSentryTestBase.java 100d885 sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/TestAuthorize.java 53f1f47 Diff: https://reviews.apache.org/r/63308/diff/2/ Changes: https://reviews.apache.org/r/63308/diff/1-2/ Testing ------- Made sure the e2e tests ae passing. Thanks, kalyan kumar kalvagadda