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

Introduce SkipBlocksWithOutOfOrderChunksEnabled feature #4707

Merged

Conversation

alvinlin123
Copy link
Member

@alvinlin123 alvinlin123 commented Apr 6, 2022

Signed-off-by: Alvin Lin alvinlin@amazon.com

What this PR does:
This PR allows user to configure if they want to skip out of order chunks during compaction (special thanks to @alanprot working on #4706 concurrently).

This PR also allows manual creation of no-compact-mark.json.

Which issue(s) this PR fixes:
Fixes #4692
Partially addresses #4705 - this change will allow manual creation of no-compact-mark.json

Checklist

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

@alvinlin123 alvinlin123 marked this pull request as ready for review April 6, 2022 09:52
@@ -2,533 +2,534 @@

## master / unreleased

* [FEATURE] Ruler: Add `external_labels` option to tag all alerts with a given set of labels.
Copy link
Member

Choose a reason for hiding this comment

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

seems that those changes were not intentional

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh this one is intentional because FEATURE should go after CHANGE (I as added the PR number)

Copy link
Member Author

Choose a reason for hiding this comment

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

I will fix the unintended change (from '*' to '-' four bullets)

Signed-off-by: Alvin Lin <alvinlin@amazon.com>
Signed-off-by: Alvin Lin <alvinlin@amazon.com>
Signed-off-by: Alvin Lin <alvinlin@amazon.com>
Signed-off-by: Alvin Lin <alvinlin@amazon.com>
Signed-off-by: Alvin Lin <alvinlin@amazon.com>
Signed-off-by: Alvin Lin <alvinlin@amazon.com>
Signed-off-by: Alvin Lin <alvinlin@amazon.com>
Signed-off-by: Alvin Lin <alvinlin@amazon.com>
@alanprot
Copy link
Member

alanprot commented Apr 6, 2022

LGTM

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.

Compactor - skipping out-of-order blocks feature is disabled
2 participants