Skip to content

Commit

Permalink
Udpate db_universal_compaction_test according to PR facebook#3970 (fa…
Browse files Browse the repository at this point in the history
…cebook#3995)

Summary:
The SST file sizes changed slightly after the improvement of PR facebook#3970
which reduces the size of the properties block. Before PR facebook#3970 a size
ratio compaction included all of the first four flushed files but it
only includes two files after. We increase the size_ratio universal
compaction option to make that compaction include all four files again.
Closes facebook#3995

Differential Revision: D8426925

Pulled By: fgwu

fbshipit-source-id: 1429c38672e9f4fb4d4881fd4b06db45c4861d62
  • Loading branch information
fgwu authored and rcane committed Sep 13, 2018
1 parent 878fd61 commit 89b7e45
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion db/db_universal_compaction_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1363,7 +1363,7 @@ TEST_P(DBTestUniversalCompaction, UniversalCompactionCFPathUse) {
options.memtable_factory.reset(
new SpecialSkipListFactory(KNumKeysByGenerateNewFile - 1));
options.compaction_style = kCompactionStyleUniversal;
options.compaction_options_universal.size_ratio = 5;
options.compaction_options_universal.size_ratio = 10;
options.write_buffer_size = 111 << 10; // 114KB
options.arena_block_size = 4 << 10;
options.level0_file_num_compaction_trigger = 2;
Expand Down

0 comments on commit 89b7e45

Please sign in to comment.