Skip to content

Retry ExceptionHandler could be set through ServiceOptions #1289

@Horneth

Description

@Horneth

The StorageImpl class and others use the EXCEPTION_HANDLER defined in the BaseService class when invoking runWithRetries.

The RetryParams and Clock can be set in the ServiceOptions and will be used when invoking runWithRetries, however the exception handler seems hardcoded.

Is that a definitive decision or would it be possible to have the exception handler also be passed through the options ?

Why

In order to be particularly resilient to transient errors it would be beneficial to be able to retry on a larger set of exceptions.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions