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

Partitioning compaction for Cortex #5465

Closed

Conversation

alexqyle
Copy link
Contributor

@alexqyle alexqyle commented Jul 18, 2023

What this PR does:
This implements #4843.

The main logic of this implementation is described inside Timeseries Partitioning in Compactor Proposal. Basically, the grouper takes in all blocks belongs to certain compaction time range and partitions them into sub-groups (partitions) each partition contains subset of incoming resource blocks. Then sends partitions for compaction. The implementation supports multiple compactors working on different partitions in parallel so source blocks with earlier time range will not be delayed because of partitioning.

Which issue(s) this PR fixes:
Fixes #4705

Checklist

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

Signed-off-by: Alex Le <[email protected]>
@alexqyle alexqyle marked this pull request as ready for review July 19, 2023 00:22
alexqyle and others added 26 commits July 25, 2023 12:48
…h partition and partition info is missing

Signed-off-by: Alex Le <[email protected]>
Signed-off-by: Alex Le <[email protected]>
Signed-off-by: Alex Le <[email protected]>
Signed-off-by: Alex Le <[email protected]>
Signed-off-by: Alex Le <[email protected]>
Signed-off-by: Alex Le <[email protected]>
Signed-off-by: Alex Le <[email protected]>
… being deleted when there is missing sample in result blocks.

Signed-off-by: Alex Le <[email protected]>
Signed-off-by: Alex Le <[email protected]>
… cleaner. Add jitter to beginning of compaction iteration.

Signed-off-by: Alex Le <[email protected]>
@alexqyle alexqyle closed this Jul 11, 2024
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.

Failure to compact due to maximum index size 64 GiB
1 participant