From 93cc4cdd0d27ba83d5f38fae7eb18cc396d05a45 Mon Sep 17 00:00:00 2001 From: Yu Zhang Date: Tue, 10 Sep 2024 11:00:51 -0700 Subject: [PATCH] Test this API only in ROCKSDB_ASSERT_STATUS_CHECKED mode --- db/memtable_list.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/db/memtable_list.cc b/db/memtable_list.cc index a4ad1de5719..c3612656e24 100644 --- a/db/memtable_list.cc +++ b/db/memtable_list.cc @@ -583,7 +583,7 @@ Status MemTableList::TryInstallMemtableFlushResults( // TODO(myabandeh): Not sure how batch_count could be 0 here. if (batch_count > 0) { VersionEdit edit; -#ifndef NDEBUG +#ifdef ROCKSDB_ASSERT_STATUS_CHECKED if (memtables_to_flush.size() == memlist.size()) { // TODO(yuzhangyu): remove this testing code once the // `GetEditForDroppingCurrentVersion` API is used by the atomic data @@ -598,7 +598,7 @@ Status MemTableList::TryInstallMemtableFlushResults( #else edit = GetDBRecoveryEditForObsoletingMemTables( vset, *cfd, edit_list, memtables_to_flush, prep_tracker); -#endif /* !NDEBUG */ +#endif // ROCKSDB_ASSERT_STATUS_CHECKED TEST_SYNC_POINT_CALLBACK( "MemTableList::TryInstallMemtableFlushResults:" "AfterComputeMinWalToKeep",