From ebb5dafef0d93e5958ad2d1db2d64a4a7983b8bf Mon Sep 17 00:00:00 2001 From: "Raphael S. Carvalho" Date: Tue, 11 Jul 2017 11:02:40 -0300 Subject: [PATCH] lcs: remove useless filter for demotion procedure there's no way a sstable from a level higher than N+1 will be in set of candidates that can be either level N or level N + 1. Signed-off-by: Raphael S. Carvalho Message-Id: <20170711140241.11023-1-raphaelsc@scylladb.com> --- sstables/leveled_manifest.hh | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/sstables/leveled_manifest.hh b/sstables/leveled_manifest.hh index a18438bffd..41738cfa63 100644 --- a/sstables/leveled_manifest.hh +++ b/sstables/leveled_manifest.hh @@ -316,15 +316,8 @@ private: auto r = ::range::make(sstable->get_first_decorated_key(), sstable->get_last_decorated_key()); if (boundaries.contains(r, dht::ring_position_comparator(*_schema))) { logger.info("Adding high-level (L{}) {} to candidates", sstable->get_sstable_level(), sstable->get_filename()); - - auto result = std::find_if(std::begin(candidates), std::end(candidates), [&sstable] (auto& candidate) { - return sstable->generation() == candidate->generation(); - }); - if (result != std::end(candidates)) { - continue; - } candidates.push_back(sstable); - return candidates; + break; } } return candidates;