fix: thread UserComparator through Run, KeyRange, and Version methods#117

Merged

polaz merged 24 commits into

mainfrom

feat/#98-bug-runpush-sorts-tables-lexicographically-ignorin

Mar 23, 2026

Commits on Mar 23, 2026

fix: thread UserComparator through Run, KeyRange, and Version methods
polaz
committed
fix: use comparator-aware RunReader in point-read merge path
polaz
committed
fix: use monotonic seqnos in leveled compaction test
polaz
committed
fix: resolve CI lint errors
polaz
committed
fix: use comparator-aware containment in drop_range strategy
polaz
committed
fix(compaction): allow multi-run levels in L1+ debug assert
polaz
committed
fix(run_reader): keep RunReader::new in public API, delegate to new_cmp
polaz
committed
fix(compaction): skip L1+ pick when levels are multi-run
polaz
committed
fix(compaction): restore relaxed debug_assert for multi-run levels
polaz
committed
fix(run_reader): gate dead_code expect behind cfg(not(test))
polaz
committed
fix(run_reader): qualify doc link and correct dead_code reason
polaz
committed
refactor(run): extract shared trim_slice helper from get_contained
polaz
committed
docs(key_range,leveled): correct contains_range_cmp reference and expect message
polaz
committed
refactor(compaction): replace debug_assert with log::debug for multi-run levels
polaz
committed
docs(version): clarify recovery preserves comparator-sorted run order
polaz
committed
docs(test): clarify ignore reason for reverse comparator range scans
polaz
committed
test(run): add unit test for get_contained_cmp with reverse comparator
polaz
committed
docs(compaction): link multi-run picker concern to #122 Part 3
polaz
committed
perf(compaction): make pick_minimal_compaction multi-run aware
polaz
committed
fix(test): use comparator-ordered key ranges in reverse cmp fixture
polaz
committed
fix(compaction): scope take_while per-run in multi-run picker
polaz
committed
test(comparator): add multi-table run range scan with u64 comparator
polaz
committed
docs(run): clarify trim_slice returns span between first and last match
polaz
committed
docs(table): clarify aggregate_run_key_range is comparator-agnostic
polaz
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: thread UserComparator through Run, KeyRange, and Version methods#117

fix: thread UserComparator through Run, KeyRange, and Version methods#117
polaz merged 24 commits into
mainfrom
feat/#98-bug-runpush-sorts-tables-lexicographically-ignorin

Commits on Mar 23, 2026

fix: thread UserComparator through Run, KeyRange, and Version methods

fix: use comparator-aware RunReader in point-read merge path

fix: use monotonic seqnos in leveled compaction test

fix: resolve CI lint errors

fix: use comparator-aware containment in drop_range strategy

fix(compaction): allow multi-run levels in L1+ debug assert

fix(run_reader): keep RunReader::new in public API, delegate to new_cmp

fix(compaction): skip L1+ pick when levels are multi-run

fix(compaction): restore relaxed debug_assert for multi-run levels

fix(run_reader): gate dead_code expect behind cfg(not(test))

fix(run_reader): qualify doc link and correct dead_code reason

refactor(run): extract shared trim_slice helper from get_contained

docs(key_range,leveled): correct contains_range_cmp reference and expect message

refactor(compaction): replace debug_assert with log::debug for multi-run levels

docs(version): clarify recovery preserves comparator-sorted run order

docs(test): clarify ignore reason for reverse comparator range scans

test(run): add unit test for get_contained_cmp with reverse comparator

docs(compaction): link multi-run picker concern to #122 Part 3

perf(compaction): make pick_minimal_compaction multi-run aware

fix(test): use comparator-ordered key ranges in reverse cmp fixture

fix(compaction): scope take_while per-run in multi-run picker

test(comparator): add multi-table run range scan with u64 comparator

docs(run): clarify trim_slice returns span between first and last match

docs(table): clarify aggregate_run_key_range is comparator-agnostic