amzn / smoke-http

Specialised HTTP Client for service operations abstracted from the HTTP protocol.
Apache License 2.0
71 stars 15 forks source link

Use regular retry loop when aborted attempts are exhausted #137

Closed pbthif closed 9 months ago

pbthif commented 9 months ago

Issue #, if available:

Description of changes: HTTP errors detected as aborted attempts are retried very quickly (2 ms), without exponential backoff. When those errors are caused by connection issues that are transient, but still take more than 10 ms to resolve, calls will fail. With this change, we would transition to using the regular retry loop (with exponential backoff) after the retry-fast attempts are exhausted.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.