-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63896/
-----------------------------------------------------------
(Updated Nov. 17, 2017, 10:29 p.m.)
Review request for sentry.
Bugs: sentry-2052
https://issues.apache.org/jira/browse/sentry-2052
Repository: sentry
Description
-------
This patch has a couple of improvements:
* Set the transaction retry max to 1 instead of 10. This high number was
causing much of the overhead on the tests. I could not set to 0 because the
TestSentryStore takes more time for some reason.
* Moved the SentryStore object creation from the @Before method to the
@BeforeClass method. This also save precious 35 seconds.
With the patch the time is reduced from 3min to 45 seconds.
Diffs (updated)
-----
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
cf83e7796fd5bc22c76cba6096b0a821e82b87a7
Diff: https://reviews.apache.org/r/63896/diff/2/
Changes: https://reviews.apache.org/r/63896/diff/1-2/
Testing
-------
Thanks,
Sergio Pena