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

fix: speed up cloud linter command #306

Closed
wants to merge 2 commits into from
Closed

Conversation

bruuuuuuuce
Copy link
Contributor

This command just speeds up the elasticache/getMetrics api calls. We need to refactor a bit how the limiter is working. We should be able to query DescribeDynamodbTables at a higher rate that we query the DescribeTimeToLive api. I spent a bit of time trying to configure separate rate limiters for the ddb api calls, but it was not behaving as expected. I think we might need to first describe all of the ddb tables, maybe have some heuristics around which tables are worth attempting to get a ttl off of, and then call those with a separate slower rate limiter

Before
Screen Shot 2024-04-10 at 6 15 17 PM

After
image

@bruuuuuuuce
Copy link
Contributor Author

I'm working on adding a heuristic around querying the time to live metric, which will supersede this pr

@bruuuuuuuce
Copy link
Contributor Author

closing in favor of #307

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant