book/rocksdb
1
0
Fork 0
mirror of https://github.com/facebook/rocksdb.git synced 2025-06-03 02:55:55 +08:00
Code Issues Packages Projects Releases Wiki Activity

Add flush_job_test to the list of ASSERT_STATUS_CHECKED tests (#7445)

Browse Source 
Summary: Pull Request resolved: https://github.com/facebook/rocksdb/pull/7445

Test Plan: pass ASSERT_STATUS_CHECKED=1 make -j48 flush_job_test

Reviewed By: akankshamahajan15

Differential Revision: D23969372

Pulled By: zhichao-cao

fbshipit-source-id: 498ff45ef84e07ec27a8f35d0874d3371412afe9
This commit is contained in:
Zhichao Cao 2020-09-28 14:57:48 -07:00 committed by Facebook GitHub Bot
parent 1abbc56aba
commit d71cfe04e4
3 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
Makefile
View File

@ -641,6 +641,7 @@ ifdef ASSERT_STATUS_CHECKED
io_tracer_test \
merge_helper_test \
memtable_list_test \
flush_job_test \
ifeq ($(USE_FOLLY_DISTRIBUTED_MUTEX),1)
TESTS_PASSING_ASC += folly_synchronization_distributed_mutex_test

1
db/flush_job.cc
View File

@ -130,6 +130,7 @@ FlushJob::FlushJob(
}
FlushJob::~FlushJob() {
io_status_.PermitUncheckedError();
ThreadStatusUtil::ResetThreadStatus();
}

1
db/flush_job_test.cc
View File

@ -111,6 +111,7 @@ class FlushJobTest : public testing::Test {
ASSERT_OK(s);
// Make "CURRENT" file that points to the new manifest file.
s = SetCurrentFile(fs_.get(), dbname_, 1, nullptr);
ASSERT_OK(s);
}
Env* env_;