Skip to content

Commit

Permalink
compaction_manager: run_offstrategy_compaction: retrieve owned_ranges…
Browse files Browse the repository at this point in the history
… from compaction_state

perform_offstrategy is called from try_perform_cleanup
when there are sstables in the maintenance set that require
cleanup.  In this case the compaction_state.owned_ranges_ptr
is set, but run_offstrategy_compaction currently ignores
that and doesn't piggyback cleanup on reshape, which
it must.

Fixes scylladb#15041

Signed-off-by: Benny Halevy <bhalevy@scylladb.com>
  • Loading branch information
bhalevy committed Aug 14, 2023
1 parent b120d35 commit 81dd87b
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions compaction/compaction_manager.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1308,6 +1308,7 @@ class offstrategy_compaction_task_executor : public compaction_task_executor, pu
new_unused_sstables.insert(sst);
return sst;
};
desc->owned_ranges = _compaction_state.owned_ranges_ptr;
auto input = boost::copy_range<std::unordered_set<sstables::shared_sstable>>(desc->sstables);

sstables::compaction_result ret;
Expand Down

0 comments on commit 81dd87b

Please sign in to comment.