Skip to content

feat: added caching for the BYO Catalog#949

Merged
ElderMatt merged 16 commits intomainfrom
APL-1662
Mar 12, 2026
Merged

feat: added caching for the BYO Catalog#949
ElderMatt merged 16 commits intomainfrom
APL-1662

Conversation

@ElderMatt
Copy link
Contributor

This PR adds caching for the BYO catalog.

  • On startup of the api, all the catalog charts get retrieved
  • Every 10 minutes, all the catalog charts get retrieved
  • On Catalog create, all the charts for that specific catalog are retrieved
  • On Catalog update, all the charts for that specific catalog are retrieved
  • On Catalog delete, all the charts for that specific catalog are deleted

New API endpoints:

  • RefreshCatalogCache: can be used to manually trigger the refresh of the catalog cache

@ElderMatt ElderMatt marked this pull request as ready for review March 10, 2026 14:10
Copy link
Collaborator

@ferruhcihan ferruhcihan left a comment

Choose a reason for hiding this comment

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

Tested and it works. LGTM overall, but there are a few minor edge cases that might cause issues, even if they are rare.

Copy link
Collaborator

@ferruhcihan ferruhcihan left a comment

Choose a reason for hiding this comment

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

Tested and LGTM

@ElderMatt ElderMatt merged commit b96f3df into main Mar 12, 2026
9 checks passed
@ElderMatt ElderMatt deleted the APL-1662 branch March 12, 2026 09:11
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.

4 participants