diff options
author | Andrew Kryczka <ajkr@users.noreply.github.com> | 2018-09-18 12:06:59 -0700 |
---|---|---|
committer | Facebook Github Bot <facebook-github-bot@users.noreply.github.com> | 2018-09-18 12:13:20 -0700 |
commit | 990b52e95b3b2d7bac72ef468a3f5665497145c3 (patch) | |
tree | 9b18c9330779e0d30f2b483a977300254ec5ebba /db/range_del_aggregator.cc | |
parent | 27221b0cc29466923bc060122e861a862b3e51ab (diff) |
Unit test for custom comparator RangeDelAggregator (#4388)
Summary:
Add a unit test for range collapsing when non-default comparator is used. This exposes the bug fixed in #4386.
Pull Request resolved: https://github.com/facebook/rocksdb/pull/4388
Differential Revision: D9918252
Pulled By: ajkr
fbshipit-source-id: 99501b96b251eab41791a7e33b27055ee36c5c39
Diffstat (limited to 'db/range_del_aggregator.cc')
-rw-r--r-- | db/range_del_aggregator.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/db/range_del_aggregator.cc b/db/range_del_aggregator.cc index e343cf611..1854377e4 100644 --- a/db/range_del_aggregator.cc +++ b/db/range_del_aggregator.cc @@ -173,7 +173,7 @@ class CollapsedRangeDelMap : public RangeDelMap { const Comparator* ucmp_; public: - CollapsedRangeDelMap(const Comparator* ucmp) + explicit CollapsedRangeDelMap(const Comparator* ucmp) : rep_(stl_wrappers::LessOfComparator(ucmp)), ucmp_(ucmp) { InvalidatePosition(); |