From cc253982ddb155475a0b5d3c908847eab30bc53f Mon Sep 17 00:00:00 2001 From: Andrew Kryczka Date: Thu, 2 Mar 2017 10:45:04 -0800 Subject: [PATCH] Use more default options in db_bench Summary: The default behavior was too weird because, previously, we got the L0 file size limit (64MB) from Options default and L1+ file size limit (2MB) from the hardcoded value. We should get both from Options default. Closes https://github.com/facebook/rocksdb/pull/1943 Differential Revision: D4640301 Pulled By: ajkr fbshipit-source-id: fd8c0fd --- tools/db_bench_tool.cc | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/tools/db_bench_tool.cc b/tools/db_bench_tool.cc index 31af174fe..1e7030059 100644 --- a/tools/db_bench_tool.cc +++ b/tools/db_bench_tool.cc @@ -490,12 +490,16 @@ DEFINE_string(wal_dir, "", "If not empty, use the given dir for WAL"); DEFINE_int32(num_levels, 7, "The total number of levels"); -DEFINE_int64(target_file_size_base, 2 * 1048576, "Target file size at level-1"); +DEFINE_int64(target_file_size_base, rocksdb::Options().target_file_size_base, + "Target file size at level-1"); -DEFINE_int32(target_file_size_multiplier, 1, +DEFINE_int32(target_file_size_multiplier, + rocksdb::Options().target_file_size_multiplier, "A multiplier to compute target level-N file size (N >= 2)"); -DEFINE_uint64(max_bytes_for_level_base, 10 * 1048576, "Max bytes for level-1"); +DEFINE_uint64(max_bytes_for_level_base, + rocksdb::Options().max_bytes_for_level_base, + "Max bytes for level-1"); DEFINE_bool(level_compaction_dynamic_level_bytes, false, "Whether level size base is dynamic");