Skip to content

Handle connection errors that occur after heavy consumption #12

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Jan 9, 2025
Merged

Handle connection errors that occur after heavy consumption #12

merged 5 commits into from
Jan 9, 2025

Conversation

@GratienDSX
Copy link
Collaborator

@GratienDSX GratienDSX commented Jan 8, 2025 •
edited
Loading

  • Catch connection errors (after a certain number of call, we receive connection error coming from Meteo-France. Waiting ~1min solves them)
  • Modified retry strategy to increase wait time between each attempt
  • After reading the grib file, remove its parent directory if empty

@GratienDSX GratienDSX changed the title Handle connection errors Handle connection errors that occur after heavy consumption Jan 8, 2025
ThomasBouche
ThomasBouche previously approved these changes Jan 8, 2025
View reviewed changes
develop-cs
develop-cs previously approved these changes Jan 9, 2025
View reviewed changes
@GratienDSX GratienDSX merged commit 9efd4bd into main Jan 9, 2025
4 checks passed
@GratienDSX GratienDSX deleted the feature/catch_connection_error branch January 9, 2025 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@develop-cs develop-cs develop-cs approved these changes

@ThomasBouche ThomasBouche Awaiting requested review from ThomasBouche

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@GratienDSX @develop-cs @ThomasBouche