-
Notifications
You must be signed in to change notification settings - Fork 96
Closed
Labels
priority: p3Desirable enhancement or fix. May not be included in next release.Desirable enhancement or fix. May not be included in next release.type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.‘Nice-to-have’ improvement, new feature or different behavior or design.
Description
Currently, retries only expire due to time-based timeout. In some cases, it is useful to have an attempt-based expiration option (for example, for datastore).
We could add a max_tries argument to our retry functions, and set it to None by default. But allow users to override it
This feature would have implications for #530, which would have to expose an additional failure reason instead of the bool is_timeout
Metadata
Metadata
Assignees
Labels
priority: p3Desirable enhancement or fix. May not be included in next release.Desirable enhancement or fix. May not be included in next release.type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.‘Nice-to-have’ improvement, new feature or different behavior or design.