2.26: What replaces RetryPolicy.none()?

[henricook]

@sugmanue in case you have the time to help 🙏🏻

I'm just upgraded to 2.26.0 and I had a RetryPolicy.none() - are either of these the correct solutions now?:

1. Don't attach a retryPolicy at all (does it default to no retry then?)
   or
2. Attach a retry strategy with a maxAttempts of 1 like (Scala):

  val retryStrategy: StandardRetryStrategy = StandardRetryStrategy
    .builder()
    .maxAttempts(1)
    .backoffStrategy(BackoffStrategy.retryImmediately())
    .build()
    
  private val clientConfiguration: ClientOverrideConfiguration =
    ClientOverrideConfiguration
      .builder()
      .retryStrategy(retryStrategy)
      .build()

[sugmanue]

Hi @henricook, the equivalent to RetryPolicy.none() is DefaultRetryStrategy.doNotRetry() see here, which is similar to what you have there. Notice also that using RetryPolicy.none() is, and will be for the foreseeable future, still supported.

Let us know if you have any other questions or concerns.

[henricook]

Thanks for this - I run in a no-warnings environment so have to deal with deprecations immediately (And RetryPolicy shows up as deprecated). DefaultRetryStrategy.doNotRetry() works - thanks for that info!

[github-actions[bot]]

Hello! Reopening this discussion to make it searchable.