4.x: Introduce exponential retry backoff policy #483
Conversation
dkropachev
force-pushed
the
dk/4.x-add-exponential-retry-backoff-policy
branch
5 times, most recently
from
798c52b to
ba056bb
Compare
Make driver wait before retry. It is needed to mitigate retry storms that can happen in certain cases.
dkropachev
force-pushed
the
dk/4.x-add-exponential-retry-backoff-policy
branch
from
ba056bb to
af7f1b3
Compare
There was a problem hiding this comment.
Documentation in multiple places is not adjusted to describe the new implementations. Often it refers to other existing classes or interfaces and does not list newly added params.
Is it necessary to introduce a new separate category of retry policies? I think from the user perspective it may be hard to understand how does the RetryPolicy interact with BackOffRetryPolicy.
Ideally the new retry policy should be just an implementation of RetryPolicy interface.
| /** The class of the retry policy. */ | ||
| public static final TypedDriverOption<String> BACKOFF_RETRY_POLICY_CLASS = | ||
| new TypedDriverOption<>(DefaultDriverOption.BACKOFF_RETRY_POLICY_CLASS, GenericType.STRING); | ||
| /** The class of the retry policy. */ | ||
| public static final TypedDriverOption<Integer> BACKOFF_RETRY_BASE_BACKOFF_MS = | ||
| new TypedDriverOption<>( | ||
| DefaultDriverOption.BACKOFF_RETRY_BASE_BACKOFF_MS, GenericType.INTEGER); | ||
| /** The class of the retry policy. */ | ||
| public static final TypedDriverOption<Integer> BACKOFF_RETRY_MAX_BACKOFF_MS = | ||
| new TypedDriverOption<>( | ||
| DefaultDriverOption.BACKOFF_RETRY_MAX_BACKOFF_MS, GenericType.INTEGER); | ||
| /** The class of the retry policy. */ | ||
| public static final TypedDriverOption<Double> BACKOFF_RETRY_JITTER_RATIO = | ||
| new TypedDriverOption<>(DefaultDriverOption.BACKOFF_RETRY_JITTER_RATIO, GenericType.DOUBLE); |
There was a problem hiding this comment.
Descriptions here look incorrect
It is definitely easier, I will try to make it part of regular retry |
Make driver wait before retry.
It is needed to mitigate retry storms that can happen in certain cases.
Fixes: #481