andreacavalli/rocksdb
0
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix DBSSTest::AddExternalSstFileSkipSnapshot valgrind fail

Browse Source 
Summary: Fix the test by releasing the last snapshot

Test Plan: run the test under valgrind

Reviewers: andrewkr, yiwu, lightmark, sdong

Reviewed By: sdong

Subscribers: andrewkr, dhruba

Differential Revision: https://reviews.facebook.net/D62091
This commit is contained in:
Islam AbdelRahman 2016-08-15 14:04:40 -07:00
parent dd7a748cff
commit 64a0082c69
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
db/db_sst_test.cc
View File

@ -1345,6 +1345,8 @@ TEST_F(DBSSTTest, AddExternalSstFileSkipSnapshot) {
ASSERT_OK(db_->AddFile(std::vector<ExternalSstFileInfo>(1, file3_info), false, true));
ASSERT_EQ(Get(Key(300)), Key(300) + ("_val"));
ASSERT_EQ(Get(Key(300), s2), Key(300) + ("_val"));
db_->ReleaseSnapshot(s2);
}
TEST_F(DBSSTTest, AddExternalSstFileMultiThreaded) {