dolysis/rocksdb
3
0
Fork 0
mirror of https://github.com/facebook/rocksdb.git synced 2024-11-26 16:30:56 +00:00
Code Issues Packages Projects Releases Wiki Activity

Blob DB: Fix BlobDBTest::SnapshotAndGarbageCollection asan failure

Browse source 
Summary:
Fix unreleased snapshot at the end of the test.
Closes https://github.com/facebook/rocksdb/pull/3126

Differential Revision: D6232867

Pulled By: yiwu-arbug

fbshipit-source-id: 651ca3144fc573ea2ab0ab20f0a752fb4a101d26
This commit is contained in:
Yi Wu 2017-11-03 10:22:50 -07:00 committed by Facebook Github Bot
parent 24ad430600
commit 2581c0a5a1
1 changed files with 1 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
utilities/blob_db/blob_db_test.cc
View file

@ -883,6 +883,7 @@ TEST_F(BlobDBTest, SnapshotAndGarbageCollection) {
if (i == 0 || i == 3 || (i == 2 && delete_key)) { if (i == 0 || i == 3 || (i == 2 && delete_key)) {
// The snapshot shouldn't see data in bfile // The snapshot shouldn't see data in bfile
ASSERT_EQ(num_files - 1, blob_db_impl()->TEST_GetBlobFiles().size()); ASSERT_EQ(num_files - 1, blob_db_impl()->TEST_GetBlobFiles().size());
blob_db_->ReleaseSnapshot(snapshot);
} else { } else {
// The snapshot will see data in bfile, so the file shouldn't be deleted // The snapshot will see data in bfile, so the file shouldn't be deleted
ASSERT_EQ(num_files, blob_db_impl()->TEST_GetBlobFiles().size()); ASSERT_EQ(num_files, blob_db_impl()->TEST_GetBlobFiles().size());