-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65268/
-----------------------------------------------------------
Review request for sentry, Arjun Mishra, Na Li, Sergio Pena, and Vadim Spector.
Bugs: SENTRY-1904
https://issues.apache.org/jira/browse/SENTRY-1904
Repository: sentry
Description
-------
The TransactionManager uses exponential backoff strategy for transaction
retries. This may cause some transactions to be delayed by a very long time. We
should also have a constraint on the max time for a transaction so that we do
not retry for too long.
New configuration parameter that limits the max time spent by transaction retry
is added. When the next retry interval exceeds the configured max time,
transaction should not retry and return a failure.
Diffs
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/HMSFollower.java
2f2b98412e7dfdcc847ffe7975a70f452554e747
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
edea5b64d8f98c93aafc1fe43fa97e00c2ce2948
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/TransactionManager.java
f4ff962a67f8a5c23cc5c8daa7bcb861d2e6b6a5
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
7e02874b4be6a7109108809b1f404fe971b7b8e2
Diff: https://reviews.apache.org/r/65268/diff/1/
Testing
-------
Made sure all the tests pass.
Thanks,
kalyan kumar kalvagadda
