From 3f9b75604d995a626b49f2d2d9fada5ec176fe8e Mon Sep 17 00:00:00 2001 From: Cheng Chang Date: Thu, 3 Sep 2020 23:16:06 -0700 Subject: [PATCH] Fix wrong level args (#7346) Summary: The level args should be output level instead of input levels. Pull Request resolved: https://github.com/facebook/rocksdb/pull/7346 Test Plan: make check Reviewed By: ajkr Differential Revision: D23506373 Pulled By: cheng-chang fbshipit-source-id: b2f701d44c13581c5c10c4dbebded4fcd354d641 --- db/compaction/compaction_picker_universal.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/db/compaction/compaction_picker_universal.cc b/db/compaction/compaction_picker_universal.cc index 387fd56aa..30bbd406c 100644 --- a/db/compaction/compaction_picker_universal.cc +++ b/db/compaction/compaction_picker_universal.cc @@ -1033,9 +1033,9 @@ Compaction* UniversalCompactionBuilder::PickCompactionToOldest( MaxFileSizeForLevel(mutable_cf_options_, output_level, kCompactionStyleUniversal), LLONG_MAX, path_id, - GetCompressionType(ioptions_, vstorage_, mutable_cf_options_, start_level, - 1, true /* enable_compression */), - GetCompressionOptions(mutable_cf_options_, vstorage_, start_level, + GetCompressionType(ioptions_, vstorage_, mutable_cf_options_, + output_level, 1, true /* enable_compression */), + GetCompressionOptions(mutable_cf_options_, vstorage_, output_level, true /* enable_compression */), /* max_subcompactions */ 0, /* grandparents */ {}, /* is manual */ false, score_, false /* deletion_compaction */, compaction_reason);