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

Pre-Populate Sprint Cache To Address Rate Limits #16

Merged
merged 2 commits into from
Feb 22, 2022

Conversation

kapil1garg
Copy link
Contributor

This PR provides a hotfix for issue #13, where the Studio API would hit a rate limit for reading data from Google Spreadsheets. This was a problem when Sprint Logs are being read for the first time and the cache is empty.

To temporarily resolve this issue, this PR:

  1. Makes only 1 call per sheet to get all the data from it
  2. Retries after 60 seconds if a rate limit is hit.

I have also requested our quota for Google Sheets to be raised so that we don't have to worry as much about this issue in the future.

@kapil1garg kapil1garg merged commit c8c60d5 into main Feb 22, 2022
@kapil1garg kapil1garg deleted the issue-13/prepopulate-cache branch February 22, 2022 22:40
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.

1 participant