diff --git a/db/db_test_util.cc b/db/db_test_util.cc index 724a217bb..d3a87cb0c 100644 --- a/db/db_test_util.cc +++ b/db/db_test_util.cc @@ -118,7 +118,8 @@ bool DBTestBase::ShouldSkipOptions(int option_config, int skip_mask) { if ((skip_mask & kSkipUniversalCompaction) && (option_config == kUniversalCompaction || - option_config == kUniversalCompactionMultiLevel)) { + option_config == kUniversalCompactionMultiLevel || + option_config == kUniversalSubcompactions)) { return true; } if ((skip_mask & kSkipMergePut) && option_config == kMergePut) { diff --git a/db/db_test_util.h b/db/db_test_util.h index 0bfbcbd12..0e62fe1cf 100644 --- a/db/db_test_util.h +++ b/db/db_test_util.h @@ -700,14 +700,9 @@ class DBTestBase : public testing::Test { kBlockBasedTableWithPartitionedIndex, kBlockBasedTableWithPartitionedIndexFormat3, kPartitionedFilterWithNewTableReaderForCompactions, - + kUniversalSubcompactions, // This must be the last line kEnd, - - // TODO: This option although been there for a while was disable due to a - // mistake. Enabling it makes somem tests to fail. We should enable it and - // fix the unit tests. - kUniversalSubcompactions, }; public: diff --git a/db/external_sst_file_test.cc b/db/external_sst_file_test.cc index b895c625e..d0c961a3c 100644 --- a/db/external_sst_file_test.cc +++ b/db/external_sst_file_test.cc @@ -971,7 +971,8 @@ TEST_F(ExternalSSTFileTest, OverlappingRanges) { s = DeprecatedAddFile({file_name}); auto it = true_data.lower_bound(Key(range_start)); if (option_config_ != kUniversalCompaction && - option_config_ != kUniversalCompactionMultiLevel) { + option_config_ != kUniversalCompactionMultiLevel && + option_config_ != kUniversalSubcompactions) { if (it != true_data.end() && it->first <= Key(range_end)) { // This range overlap with data already exist in DB ASSERT_NOK(s);