Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use exponential backoff for updates #23

Open
ghost opened this issue Jul 14, 2023 · 1 comment
Open

Use exponential backoff for updates #23

ghost opened this issue Jul 14, 2023 · 1 comment
Labels
enhancement New feature, enhancement or code changes, not related to defects good first issue Good for newcomers

Comments

@ghost
Copy link

ghost commented Jul 14, 2023

If an update fails, we should delay the next update exponentially, as otherwise we'll repeatedly perform updates on a fixed interval while the network or service is degraded. Additionally, we should also not wait too long after the first failure, as the issue may be ephemeral and we should not persist a degraded or stale state.

@ghost ghost added the enhancement New feature, enhancement or code changes, not related to defects label Jul 14, 2023
@ghost ghost self-assigned this Jul 14, 2023
@DXTimer
Copy link
Contributor

DXTimer commented Jul 17, 2023

Since the plugin carries out periodic syncs we should factor in the refresh rate and depending on the refresh rate it may be more prudent to abandon the current refresh cycle if it fails more than X times.

@DXTimer DXTimer unassigned ghost Aug 10, 2023
@DXTimer DXTimer added the good first issue Good for newcomers label Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature, enhancement or code changes, not related to defects good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant