Skip to content

Unify chunks and metadata cache backend to bucket cache #6829

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

Merged
merged 3 commits into from
Jun 23, 2025

Conversation

yeya24
Copy link
Contributor

@yeya24 yeya24 commented Jun 19, 2025

What this PR does:

This PR tries to unify chunks cache and metadata cache backend into single bucket cache backend. This allows to support inmemory and multi level cache for metadata cache.

It allows to extend the same config for other caching bucket type we want to support in the future. We want to add cache for Parquet labels file so similar caching bucket backend can be reused.

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@dosubot dosubot bot added the storage/blocks Blocks storage engine label Jun 19, 2025
@yeya24 yeya24 force-pushed the unify-cache-bucket branch from 65438c0 to ebfe73f Compare June 19, 2025 07:50
@yeya24 yeya24 force-pushed the unify-cache-bucket branch from ebfe73f to e9a7eea Compare June 22, 2025 23:56
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 23, 2025
@yeya24 yeya24 merged commit e551a2e into cortexproject:master Jun 23, 2025
31 of 32 checks passed
@yeya24 yeya24 deleted the unify-cache-bucket branch June 23, 2025 03:50
bogdan-at-adobe pushed a commit to bogdan-at-adobe/cortex that referenced this pull request Jun 30, 2025
…t#6829)

* unify chunks and metadata cache to bucket cache

Signed-off-by: yeya24 <[email protected]>

* update docs

Signed-off-by: Ben Ye <[email protected]>

* update changelog

Signed-off-by: Ben Ye <[email protected]>

---------

Signed-off-by: yeya24 <[email protected]>
Signed-off-by: Ben Ye <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size/L storage/blocks Blocks storage engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants