-----------------------------------------------------------
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