From ac81130fa03a21075be3e0e7d9a93eae12783d47 Mon Sep 17 00:00:00 2001 From: sdong Date: Tue, 2 Jun 2015 09:56:22 -0700 Subject: [PATCH] Fix Bug: CompactRange() doesn't change to correct level caused by using wrong level Summary: In previous change https://reviews.facebook.net/D39099 , while renaming parameters, use a wrong parameter, causing CompactRange() to compact not wrong level. Test Plan: Run "DBTest.MigrateToDynamicLevelMaxBytesBase" which failed with the patch. Reviewers: rven, yhchiang, kradhakrishnan, igor, anthony Subscribers: leveldb, dhruba Differential Revision: https://reviews.facebook.net/D39393 --- include/rocksdb/db.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/rocksdb/db.h b/include/rocksdb/db.h index f9188e051..d25c421dc 100644 --- a/include/rocksdb/db.h +++ b/include/rocksdb/db.h @@ -430,7 +430,7 @@ class DB { bool change_level = false, int target_level = -1, uint32_t target_path_id = 0) { return CompactRange(DefaultColumnFamily(), begin, end, change_level, - change_level, target_path_id); + target_level, target_path_id); } virtual Status SetOptions(ColumnFamilyHandle* column_family, const std::unordered_map& new_options) {