accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Updated] (ACCUMULO-4847) Fix Retry API
Date Wed, 14 Mar 2018 17:09:00 GMT


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

> Fix Retry API
> -------------
>                 Key: ACCUMULO-4847
>                 URL:
>             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
> 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

View raw message