Skip to content

Adding a jitter on the head compaction and changing the max head comp… #5919

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 5 commits into from
May 3, 2024

Conversation

alanprot
Copy link
Member

@alanprot alanprot commented May 2, 2024

What this PR does:
Sometimes we see latency increase when we have multiples head compaction hapening at the same time.

This PR is allowing us to configure the interval of the head compaction up to 30 min and also adding a jitter to spread the head compaction over time.

Which issue(s) this PR fixes:
Fixes #

Checklist

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

@alanprot alanprot force-pushed the head-compaction-jitter branch from 8fa8b66 to 21e82fc Compare May 2, 2024 19:03
@pull-request-size pull-request-size bot added size/S and removed size/XS labels May 2, 2024
alanprot added 2 commits May 2, 2024 12:06
Signed-off-by: alanprot <[email protected]>
Signed-off-by: alanprot <[email protected]>
@pull-request-size pull-request-size bot added size/M and removed size/S labels May 2, 2024
@alanprot alanprot enabled auto-merge (squash) May 2, 2024 19:24
Signed-off-by: alanprot <[email protected]>
@alanprot alanprot force-pushed the head-compaction-jitter branch from 361fafe to 5c924c8 Compare May 2, 2024 21:02
Signed-off-by: alanprot <[email protected]>
@alanprot alanprot disabled auto-merge May 2, 2024 22:14
@alanprot alanprot enabled auto-merge (squash) May 2, 2024 22:14
@alanprot alanprot merged commit 04cc770 into cortexproject:master May 3, 2024
16 checks passed
@yeya24 yeya24 mentioned this pull request Oct 23, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants