[
https://issues.apache.org/jira/browse/SENTRY-1904?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16338096#comment-16338096
]
Alexander Kolbasov commented on SENTRY-1904:
--------------------------------------------
The latest patch seems to contain unrelated changes.
As for the approach - I think that if we use max time we should stop using
number of retries as a criteria and just use total time. Using both doesn't
make much sense.
> TransactionManager should limit the max time spent by transaction retry
> -----------------------------------------------------------------------
>
> Key: SENTRY-1904
> URL: https://issues.apache.org/jira/browse/SENTRY-1904
> Project: Sentry
> Issue Type: Bug
> Components: Sentry
> Affects Versions: 2.0.0
> Reporter: Alexander Kolbasov
> Assignee: kalyan kumar kalvagadda
> Priority: Major
> Attachments: SENTRY-1904.001.patch, SENTRY-1904.002.patch,
> SENTRY-1904.003.patch
>
>
> 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.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
