bahman/scylla
1
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

compaction: switch from boost::accumulate to std::views::join

Browse Source 
Replace boost::accumulate() with the standard library's alternatives
to reduce external dependencies and simplify the codebase. This
change eliminates the requirement for boost::range and makes the
implementation more maintainable.

Signed-off-by: Kefu Chai <kefu.chai@scylladb.com>

Closes scylladb/scylladb#22856
This commit is contained in:
Kefu Chai
2025-02-14 11:04:41 +08:00
committed by Botond Dénes
parent aba4d07c62
commit 9c5155fa63
1 changed files with 4 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
compaction/incremental_compaction_strategy.cc
View File

@@ -11,9 +11,6 @@
#include "compaction_manager.hh"
#include "incremental_compaction_strategy.hh"
#include "incremental_backlog_tracker.hh"
#include <boost/range/numeric.hpp>
#include <boost/range/algorithm.hpp>
#include <boost/range/adaptors.hpp>
#include <ranges>
namespace sstables {
@@ -417,11 +414,10 @@ int64_t incremental_compaction_strategy::estimated_pending_compactions(table_sta
std::vector<shared_sstable>
incremental_compaction_strategy::runs_to_sstables(std::vector<frozen_sstable_run> runs) {
return boost::accumulate(runs, std::vector<shared_sstable>(), [&] (std::vector<shared_sstable>&& v, const frozen_sstable_run& run_ptr) {
auto& run = *run_ptr;
v.insert(v.end(), run.all().begin(), run.all().end());
return std::move(v);
});
return runs
| std::views::transform([] (auto& run) -> auto& { return run->all(); })
| std::views::join
| std::ranges::to<std::vector>();
}
std::vector<frozen_sstable_run>