[jira] [Updated] (ACCUMULO-4847) Fix Retry API

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/ACCUMULO-4847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

ASF GitHub Bot updated ACCUMULO-4847:
-------------------------------------
    Labels: pull-request-available  (was: )

> Fix Retry API
> -------------
>
>                 Key: ACCUMULO-4847
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4847
>             Project: Accumulo
>          Issue Type: Bug
>          Components: fate
>            Reporter: Christopher Tubbs
>            Assignee: Christopher Tubbs
>            Priority: Blocker
>              Labels: pull-request-available
>             Fix For: 1.7.4, 1.9.0, 2.0.0
>
>
> ACCUMULO-4834 exposed a merge bug from ACCUMULO-4777. ACCUMULO-4777 added a parameter to the two Retry constructors, but since all the parameters are longs, this masked what should have been a compile failure by causing calls to the longer constructor to now use the shorter one.
> The side effect is that any calls with "unlimited number of retries", would now be interpreted as "unlimited wait time between retries"... causing the first failure to sleep for Long.MAX_LONG millis before retrying.
> I propose a refactor with a builder so the parameters cannot be confused for one another, since they all have the same long type. This is internal code, so there is no public API change.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)