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

Reply via email to