Fixed overflow bugs in backoff strategy

[Quanzzzz]

Fixed issue #1453 based on the proposed fix code provided by OlegZhukov, set up ceils for baseDelay, maxBackoffTime and retriesAttempted to prevent potential overflow problems.

Description

Added three ceils for baseDelay, maxBackoffTime and retriesAttempted respectively. Restricted the exponentialDelayMillis from overflow via applying these ceils in calculation. Added unit tests for EqualJitterBackoffStrategy.java based on proposed test cases provided by OlegZhukov.

Motivation and Context

Before this modification, the exponentialDelay has a risk of overflow due to its type as Integer. Besides, baseDelay and maxBackoffTime are not checked if they will meet overflow problems while converting into Millis. So to fix these potential problems, we need to add some restrictions on these variables, as well as the calculation.

The open issue corresponding to this pull request is: #1453

Testing

Only one test class is added, and it only includes unit tests for EqualJitterBackoffStrategyTest.java. For FullJitterBackoffStrategy.java, there isn't appropriate test cases since the exponentialDelay calculated with Full Jitter Backoff Strategy is a totally random number, which isn't suitable for testing.

The unit tests added basically covering all possible cases include: normal situation of retries, retries with extreme large duration as baseDelay and maxBackoffTime respectively, retries with single nanosecond as baseDelay and maxBackoffTime respectively, valid retries Attempt times, invalid retries Attempt times and retries with negative duration.

Screenshots (if appropriate)

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)

Checklist

• I have read the CONTRIBUTING document
• Local run of mvn install succeeds
• My code follows the code style of this project
• My change requires a change to the Javadoc documentation
• I have updated the Javadoc documentation accordingly
• I have read the README document
• I have added tests to cover my changes
• All new and existing tests passed
• A short description of the change has been added to the CHANGELOG
• My change is to implement 1.11 parity feature and I have updated LaunchChangelog

License

• I confirm that this pull request can be released under the Apache 2 license

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 2 Security Hotspots to review)
1 Code Smell

No Coverage information
No Duplication information

[codecov-io]

Codecov Report

❗ No coverage uploaded for pull request base (master@66b67ab). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1668   +/-   ##
=========================================
  Coverage          ?   74.47%           
  Complexity        ?      902           
=========================================
  Files             ?      959           
  Lines             ?    30365           
  Branches          ?     2393           
=========================================
  Hits              ?    22613           
  Misses            ?     6617           
  Partials          ?     1135

Flag	Coverage Δ	Complexity Δ
#unittests	74.47% <ø> (?)	902 <ø> (?)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 66b67ab...e5edca4. Read the comment docs.