ox icon indicating copy to clipboard operation
ox copied to clipboard

Add multi-policies for retries

Open rucek opened this issue 2 years ago • 2 comments

The current retry mechanism only supports a single retry policy for a successful/erroneous result of a given operation, which can only be customized by providing a ResultPolicy that allows for re-defining success or failing fast on given errors. The limitation of this approach is that it doesn't support different retry strategies for different types of errors.

We'd like to be able to define a multi-policy, i.e. one that assigns different retry strategies to different errors, so that e.g. we retry immediately for some errors, but use exponential backoff for others.

A multi-policy could be thought of as a total function E => RetryPolicy, where E is the error type for the operation, e.g.

sealed trait Error

object Error:
  case class WithCode(code: Int) extends Error
  case object Other extends Error

val policy = MultiPolicy {
  case Error.WithCode(code) if code == 42 => RetryPolicy.immediate(3)
  case Error.WithCode(_) => RetryPolicy.delay(4, 1.second)
  case Other => RetryPolicy.backoff(5, 100.millis)
}

One thing to consider is whether we want a notion of "failing fast" within the multi-policy - since it can already be handled by a ResultPolicy(isWorthRetrying = _ => false). Allowing to define it within a multi-policy as well - although possibly useful - might introduce ambiguity.

rucek avatar Dec 04 '23 13:12 rucek

@micossow isn't that done with .andThen?

adamw avatar Jul 12 '24 10:07 adamw

@adamw .andThen is used for combining multiple schedules (for example use a schedule A for first 3 retries, then B for another 4 retries and schedule C for remaining retries) And this issue seems to be about changing RetryConfig based on error returned by an operation.

micossow avatar Jul 12 '24 10:07 micossow