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

feat: cache routes by id to speed up alerts filtering #2142

Merged
merged 2 commits into from
Aug 19, 2024

Conversation

sloanelybutsurely
Copy link
Contributor

@sloanelybutsurely sloanelybutsurely commented Aug 16, 2024

Adds read-through cache of Routes by ID and uses this for alerts route_type filter expansion.

@sloanelybutsurely sloanelybutsurely changed the title Sloane/push svupzwuqkwvy feat: cache routes by id to speed up alerts filtering Aug 16, 2024
@sloanelybutsurely sloanelybutsurely force-pushed the sloane/push-svupzwuqkwvy branch from 1101c4d to 00b4c00 Compare August 16, 2024 15:16
@sloanelybutsurely sloanelybutsurely marked this pull request as ready for review August 16, 2024 15:16
@sloanelybutsurely sloanelybutsurely requested a review from a team as a code owner August 16, 2024 15:16
@sloanelybutsurely sloanelybutsurely force-pushed the sloane/push-ryozxqwlkomu branch from a324de6 to 6b42d0d Compare August 16, 2024 15:30
@sloanelybutsurely sloanelybutsurely force-pushed the sloane/push-svupzwuqkwvy branch 2 times, most recently from 6381332 to 68ca04b Compare August 16, 2024 15:30
@sloanelybutsurely sloanelybutsurely force-pushed the sloane/push-ryozxqwlkomu branch 2 times, most recently from 5b49d56 to c31df8e Compare August 16, 2024 20:46
@sloanelybutsurely sloanelybutsurely force-pushed the sloane/push-svupzwuqkwvy branch 2 times, most recently from 8605b59 to ff8dffc Compare August 16, 2024 20:48
@sloanelybutsurely sloanelybutsurely force-pushed the sloane/push-ryozxqwlkomu branch from c31df8e to 37f4f31 Compare August 16, 2024 20:48
@sloanelybutsurely sloanelybutsurely force-pushed the sloane/push-svupzwuqkwvy branch from ff8dffc to 5a0e5bb Compare August 16, 2024 20:54
@sloanelybutsurely sloanelybutsurely force-pushed the sloane/push-ryozxqwlkomu branch 2 times, most recently from cf0b185 to 2d88e52 Compare August 16, 2024 20:56
@sloanelybutsurely sloanelybutsurely force-pushed the sloane/push-svupzwuqkwvy branch from 5a0e5bb to ea2ff7a Compare August 16, 2024 20:59
Base automatically changed from sloane/push-ryozxqwlkomu to main August 16, 2024 20:59
@sloanelybutsurely sloanelybutsurely force-pushed the sloane/push-svupzwuqkwvy branch from ea2ff7a to bf97753 Compare August 16, 2024 21:01
Copy link
Contributor

@cmaddox5 cmaddox5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚤

@sloanelybutsurely sloanelybutsurely merged commit 6f88363 into main Aug 19, 2024
11 checks passed
@sloanelybutsurely sloanelybutsurely deleted the sloane/push-svupzwuqkwvy branch August 19, 2024 14:43
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