storage: compaction alignment test
#24636
Open
+283
−3
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The test uses different segment layouts, runs compaction and compares results. Signed-off-by: Evgeny Lazin <[email protected]>
WillemKauf
changed the title
storage: compaction alignment test
Lazin
previously approved these changes
Dec 20, 2024
Lazin
reviewed
Dec 20, 2024
| compaction_config cfg, std::optional<model::offset> new_start_offset) { | ||
| return get_disk_log_impl().adjacent_merge_compact(cfg, new_start_offset); | ||
| } | ||
|
|
||
| ss::future<> disk_log_builder::apply_sliding_window_compaction( | ||
| compaction_config cfg, std::optional<model::offset> new_start_offset) { | ||
| return get_disk_log_impl().sliding_window_compact(cfg, new_start_offset); |
There was a problem hiding this comment.
sliding_window_compact returns future<bool> so it needs something like ignore_result at the end
There was a problem hiding this comment.
Oops. That's what I get for walking away before my build finishes
Pushed
WillemKauf
force-pushed
the
e_compaction_alignment_test
branch
from
939053a to
80b0f72
Compare
CI test resultstest results on build#60030
test results on build#60036
test results on build#60040
|
WillemKauf
force-pushed
the
e_compaction_alignment_test
branch
2 times, most recently
from
6a7731e to
d6833b7
Compare
Retry command for Build#60040please wait until all jobs are finished before running the slash command |
WillemKauf
commented
Dec 23, 2024
| ss::future<ss::stop_iteration> operator()(model::record_batch rb) { | ||
| static const auto translation_batches | ||
| = model::offset_translator_batch_types(); | ||
| if ( |
There was a problem hiding this comment.
A different condition than if(rb.header().type != model::record_batch_type::raft_data).
Otherwise, compaction placeholder batches may find their way into the ot_state with a gap_length of 0.
WillemKauf
force-pushed
the
e_compaction_alignment_test
branch
from
d6833b7 to
c10d572
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Minor enhancements to #24621
Backports Required
Release Notes