RetryStrategy doOnError should only be called if retryIf matches

johanhaleby / occurrent

Unintrusive Event Sourcing Library for the JVM

https://occurrent.org

120 stars 16 forks source link

RetryStrategy doOnError should only be called if retryIf matches #141

Closed johanhaleby closed 10 months ago

johanhaleby commented 11 months ago

Maybe not? Because maybe you want to log the error regardless of what you've specified in retryIf? I'll close this for now.

johanhaleby commented 11 months ago

Men man vill kanske inte ha dubbel loggning om man specar både onError och onRetry Så kanske ska det va så att onError inte kickar in om man har retries kvar. Utan enbart när man faktiskt får ett fel. Annars är det onRetry som kallas. Kanske ska dela upp i onBeforeRetry och onAfterRetry

johanhaleby commented 11 months ago

Ha olika metoder, dvs onBeforeRetry och onAfterRetry och sen anropas bara onError om det faktiskt blir ett fel (dvs efter retries är exhausted)