Update DBTestUniversalCompaction.UniversalCompactionSingleSortedRun to use max_size_amplification_percent = 0

Summary: With max_size_amplification_percent = 0 to make sure that DBTestUniversalCompaction.UniversalCompactionSingleSortedRun tests the configuration to compact to one single sorted run.

Test Plan: Run all existing tests

Reviewers: yhchiang, andrewkr, IslamAbdelRahman

Reviewed By: IslamAbdelRahman

Subscribers: leveldb, andrewkr, dhruba

Differential Revision: https://reviews.facebook.net/D60021
main
sdong 8 years ago
parent 892e9d3047
commit c4cef07f1b
  1. 4
      db/compaction_picker.cc
  2. 5
      db/db_universal_compaction_test.cc

@ -1708,14 +1708,14 @@ Compaction* UniversalCompactionPicker::PickCompactionUniversalSizeAmp(
LogToBuffer( LogToBuffer(
log_buffer, log_buffer,
"[%s] Universal: size amp not needed. newer-files-total-size %" PRIu64 "[%s] Universal: size amp not needed. newer-files-total-size %" PRIu64
"earliest-file-size %" PRIu64, " earliest-file-size %" PRIu64,
cf_name.c_str(), candidate_size, earliest_file_size); cf_name.c_str(), candidate_size, earliest_file_size);
return nullptr; return nullptr;
} else { } else {
LogToBuffer( LogToBuffer(
log_buffer, log_buffer,
"[%s] Universal: size amp needed. newer-files-total-size %" PRIu64 "[%s] Universal: size amp needed. newer-files-total-size %" PRIu64
"earliest-file-size %" PRIu64, " earliest-file-size %" PRIu64,
cf_name.c_str(), candidate_size, earliest_file_size); cf_name.c_str(), candidate_size, earliest_file_size);
} }
assert(start_index < sorted_runs.size() - 1); assert(start_index < sorted_runs.size() - 1);

@ -127,7 +127,7 @@ TEST_P(DBTestUniversalCompaction, UniversalCompactionSingleSortedRun) {
options.level0_file_num_compaction_trigger = 0; options.level0_file_num_compaction_trigger = 0;
options.compaction_options_universal.size_ratio = 10; options.compaction_options_universal.size_ratio = 10;
options.compaction_options_universal.min_merge_width = 2; options.compaction_options_universal.min_merge_width = 2;
options.compaction_options_universal.max_size_amplification_percent = 1; options.compaction_options_universal.max_size_amplification_percent = 0;
options.write_buffer_size = 105 << 10; // 105KB options.write_buffer_size = 105 << 10; // 105KB
options.arena_block_size = 4 << 10; options.arena_block_size = 4 << 10;
@ -151,6 +151,9 @@ TEST_P(DBTestUniversalCompaction, UniversalCompactionSingleSortedRun) {
dbfull()->TEST_WaitForCompact(); dbfull()->TEST_WaitForCompact();
ASSERT_EQ(NumSortedRuns(0), 1); ASSERT_EQ(NumSortedRuns(0), 1);
} }
ASSERT_OK(Put(Key(key_idx), ""));
dbfull()->TEST_WaitForCompact();
ASSERT_EQ(NumSortedRuns(0), 1);
} }
TEST_P(DBTestUniversalCompaction, OptimizeFiltersForHits) { TEST_P(DBTestUniversalCompaction, OptimizeFiltersForHits) {

Loading…
Cancel
Save