You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In the future, when we move run_forever() it might be nice to be able to pass a retry strategy to it. A good reason to move the retry module from client-base to this repo.
ExponentialBackoff is hard to do with generic functions, because it needs to know if a previous attempt succeeded, so that it can reset its backoff interval.
This is easy to do with streams, we just reset it after every incoming message. But for functions, we'd need something like a timer to reset the backoff interval, if the function hasn't failed for a certain interval, then we say it succeeded, for example.
Yeah, maybe we can have a strategy that actually takes into account for how long the task ran, so if last time it ran for a long time it waits very little to restart and if it failed immediately it waits quite a long time to restart 🤔
In the future, when we move
run_forever()
it might be nice to be able to pass a retry strategy to it. A good reason to move theretry
module fromclient-base
to this repo.Originally posted by @llucax in frequenz-floss/frequenz-sdk-python#1081 (comment)
The text was updated successfully, but these errors were encountered: