Add ttl param to the end in CompactionOptionsFIFO constructor.

with a default value of 0.
facebook · Jun 25, 2017 · 20514a9 · 20514a9
1 parent 3a90501
commit 20514a9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/include/rocksdb/advanced_options.h b/include/rocksdb/advanced_options.h
@@ -78,8 +78,8 @@ struct CompactionOptionsFIFO {
   bool allow_compaction = false;
 
   CompactionOptionsFIFO() : max_table_files_size(1 * 1024 * 1024 * 1024) {}
-  CompactionOptionsFIFO(uint64_t _max_table_files_size, uint64_t _ttl,
-                        bool _allow_compaction)
+  CompactionOptionsFIFO(uint64_t _max_table_files_size, bool _allow_compaction,
+                        uint64_t _ttl = 0)
       : max_table_files_size(_max_table_files_size),
         ttl(_ttl),
         allow_compaction(_allow_compaction) {}

diff --git a/tools/db_bench_tool.cc b/tools/db_bench_tool.cc
@@ -2865,7 +2865,7 @@ void VerifyDBFromDB(std::string& truth_db_name) {
 #ifndef ROCKSDB_LITE
     options.compaction_options_fifo = CompactionOptionsFIFO(
         FLAGS_fifo_compaction_max_table_files_size_mb * 1024 * 1024,
-        FLAGS_fifo_compaction_ttl, FLAGS_fifo_compaction_allow_compaction);
+        FLAGS_fifo_compaction_allow_compaction, FLAGS_fifo_compaction_ttl);
 #endif  // ROCKSDB_LITE
     if (FLAGS_prefix_size != 0) {
       options.prefix_extractor.reset(