EdColeman commented on PR #3422: URL: https://github.com/apache/accumulo/pull/3422#issuecomment-1559390277
This may prevent the overflow, but I was reviewing this to see if the wait / backoff factor is what is expected when this was merged. Where ``` currentWait = Math.min(maxWait, initialWait + waitIncrement); ``` should it be something like ``` currentWait = Math.min(maxWait, currentWait + waitIncrement); ``` If you have an initial wait of 10 seconds and then you want to retry every second with a back off I would expect wait times like 10, 11, 13, 17,... But it appears that the code would be like 10, 21, 33 (or something), but basically backing off by the initialWait and the back-off increment. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
