Skip to content

/ noctua

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

Jump to bottom

dbWriteTable should support a retry functionality #79

Closed
DyfanJones opened this issue Mar 19, 2020 · 2 comments
Closed

dbWriteTable should support a retry functionality #79

DyfanJones opened this issue Mar 19, 2020 · 2 comments
Assignees
@DyfanJones
Labels
bug enhancement

Comments

@DyfanJones
Copy link
Owner

@DyfanJones DyfanJones commented Mar 19, 2020

It appears that some users are coming into an issue of throttling errors. A user has found a possible solution https://community.rstudio.com/t/use-exponential-back-off-with-dbi/56870. To make it easier for users this solution should be inbuilt into the package.
DyfanJones added a commit that referenced this issue Mar 19, 2020
@DyfanJones
added helper function retry_api_call if call initially failed (#79)
d8a698c
DyfanJones added a commit that referenced this issue Mar 19, 2020
@DyfanJones
added parameters in noctua_options to configure the retry_api_call (#79)
73f03a7
DyfanJones added a commit that referenced this issue Mar 19, 2020
@DyfanJones
added retry_api_call when sending data to s3 (#79)
4ac14d5
DyfanJones added a commit that referenced this issue Mar 19, 2020
@DyfanJones
added retry_api_call when retrieving data from S3 (#79)
6aa8f67
@DyfanJones DyfanJones added the bug label Mar 19, 2020
@DyfanJones DyfanJones self-assigned this Mar 19, 2020
@DyfanJones
Copy link
Owner Author

@DyfanJones DyfanJones commented Mar 19, 2020

As unit tests aren't coming across this issue. I will have to ask for help from the author of the RStudio ticket: https://community.rstudio.com/t/use-exponential-back-off-with-dbi/56870
DyfanJones added a commit that referenced this issue Mar 24, 2020
@DyfanJones
retry api call if initially failed (#79)
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
a187676
@DyfanJones
Copy link
Owner Author

@DyfanJones DyfanJones commented Mar 31, 2020

The new retry functionality using exponential back off seems to have fixed issues highlighted by Rstudio community ticket. Will close this ticket.

If issue hasn't been resolved please reopen this ticket.
@DyfanJones DyfanJones closed this Mar 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DyfanJones DyfanJones

Labels
bug enhancement
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
@DyfanJones
You can’t perform that action at this time.