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

google spreadsheets still hitting rate limit even with caching #13

Open
kapil1garg opened this issue Feb 16, 2022 · 3 comments
Open

google spreadsheets still hitting rate limit even with caching #13

kapil1garg opened this issue Feb 16, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@kapil1garg
Copy link
Contributor

when the SprintCache is empty, the sprint log parser will still run into rate limits trying to parse all sprints in DTR (such as if queried by an orchestration script).

See if the quota can be raised, and/or retry when request fails

@kapil1garg kapil1garg added the bug Something isn't working label Feb 16, 2022
@kapil1garg
Copy link
Contributor Author

On second thought, this might be timeouts. Need to investigate

@kapil1garg
Copy link
Contributor Author

a temporary fix for this is for the studio API on start-up to populate all the data caches by itself (rather than waiting for a request to come in). then when queries are made, only the sprint logs where the data has changed will need to be fetched since everything else can be retrieved from cache (which is much faster)

@kapil1garg
Copy link
Contributor Author

kapil1garg commented Feb 22, 2022

PR #16 includes a hotfix for this solution. One limitation of using the google-spreadsheet wrapper package is its limited flexibility in how to read data from a sheet (must make a query for each worksheet). A more permanent solution might be to switch to Google's own APIs which support reading an entire spreadsheet in 1 call which would reduce our calls to 1 for every project instead of 5-6 (see here), and also supports exponential backoff so that requests are automatically re-tried.

seems like there's also a simple implementation as part of google-spreadsheet that is implemented, but not in the latest package yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant