From 8bf73208a45a8f73fdb1c49fada816835a4c5cf0 Mon Sep 17 00:00:00 2001 From: Abhishek Madan Date: Wed, 19 Dec 2018 15:43:08 -0800 Subject: [PATCH] Remove stale TODO (#4800) Summary: This TODO was already addressed, but I forgot to remove it before landing the PR it came from. Pull Request resolved: https://github.com/facebook/rocksdb/pull/4800 Differential Revision: D13522284 Pulled By: abhimadan fbshipit-source-id: 7766bc4f5b54e47d355cf26137ef5e86c604472a --- db/range_del_aggregator.cc | 2 -- 1 file changed, 2 deletions(-) diff --git a/db/range_del_aggregator.cc b/db/range_del_aggregator.cc index 3685d717d..68216fc92 100644 --- a/db/range_del_aggregator.cc +++ b/db/range_del_aggregator.cc @@ -470,8 +470,6 @@ CompactionRangeDelAggregator::NewIterator(const Slice* lower_bound, new TruncatedRangeDelMergingIter(icmp_, lower_bound, upper_bound, upper_bound_inclusive, parent_iters_)); - // TODO: add tests where tombstone fragments can be outside of upper and lower - // bound range auto fragmented_tombstone_list = std::make_shared( std::move(merging_iter), *icmp_, true /* for_compaction */,