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 RETRY instead of GET #45

Merged
merged 1 commit into from
Nov 11, 2024
Merged

Use RETRY instead of GET #45

merged 1 commit into from
Nov 11, 2024

Conversation

Moohan
Copy link
Member

@Moohan Moohan commented Nov 11, 2024

This should mean that any failed requests will be retried up to 3 times on a failure. The function will wait a random amount of time, progressively longer on each retry, so as not to overwhelm the server.

It should clear up the intermittent errors we get.

Possibly we want to use quiet = TRUE - at the moment it will print a message when it has to retry informing how long it will wait before trying again.

This should mean that any failed requests will be retried up to 3 times on a failure. The function will wait a random amount of time, progressively longer on each retry, so as not to overwhelm the server.

It should clear up the intermittent errors we get.

Possibly we want to use `quiet = TRUE` - at the moment it will print a message when it has to retry informing how long it will wait before trying again.
@Moohan Moohan requested review from csillasch and a team November 11, 2024 11:26
@csillasch csillasch merged commit b0b810c into master Nov 11, 2024
16 checks passed
@csillasch csillasch deleted the httr-RETRY branch November 11, 2024 15:35
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

Successfully merging this pull request may close these issues.

2 participants