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

Separate retry policies for transient errors and setup errors #24

Closed
istreeter opened this issue Jan 10, 2024 · 0 comments
Closed

Separate retry policies for transient errors and setup errors #24

istreeter opened this issue Jan 10, 2024 · 0 comments

Comments

@istreeter
Copy link
Collaborator

If an error is due to bad warehouse setup (e.g. insufficient permissions) then we want a long slow retry delay, and we want the loader to send helpful error messages to the monitoring webhook.

If an error is due to a transient error (e.g. server error or network error) then we want a faster retry delay, and no need to send messages to the webhook. If the app crashes due to many successive transient errors, then the exception will get tracked in Sentry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant