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

Loader: retry on target initialization #964

Closed
istreeter opened this issue Jul 3, 2022 · 0 comments
Closed

Loader: retry on target initialization #964

istreeter opened this issue Jul 3, 2022 · 0 comments
Milestone

Comments

@istreeter
Copy link
Contributor

Currently, the loader runs some initialization when it first starts up.

We have seen cases where the loader is configured with the wrong password for the warehouse. The initialization results in an exception, and the loader crashes. The orchestration continually restarts the app, and it crashes every time. Eventually the user gets temporarily locked out of the warehouse because of so many password failures.

Instead of crashing, it would be better to surround the initialization in a retry block with exponential backoff. As long as we back off sufficiently, then connection attempts will be infrequent and the user would not get locked out of the warehouse.

@istreeter istreeter added this to the 4.2.0 milestone Jul 4, 2022
@spenes spenes changed the title Loader: target initialization needs retry with exponential backoff Loader: retry on target initialization Jul 7, 2022
@spenes spenes closed this as completed in 4acf000 Jul 19, 2022
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