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

Fix asan_check failures

Browse source 
Summary: Pull Request resolved: https://github.com/facebook/rocksdb/pull/5589

Test Plan: TEST_TMPDIR=/dev/shm/rocksdb COMPILE_WITH_ASAN=1 OPT=-g make J=64 -j64 asan_check

Differential Revision: D16361081

Pulled By: anand1976

fbshipit-source-id: 09474832b9cfb318a840d4b633e22dfad105d58c
This commit is contained in:
anand76 2019-07-18 14:38:23 -07:00 committed by Facebook Github Bot
parent 3a6e83b56b
commit abd1fdddef
1 changed files with 3 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
db/import_column_family_test.cc
View file

@ -298,6 +298,7 @@ TEST_F(ImportColumnFamilyTest, ImportExportedSSTFromAnotherCF) {
ASSERT_OK(checkpoint->ExportColumnFamily(handles_[1], export_files_dir_, ASSERT_OK(checkpoint->ExportColumnFamily(handles_[1], export_files_dir_,
&metadata_ptr_)); &metadata_ptr_));
ASSERT_NE(metadata_ptr_, nullptr); ASSERT_NE(metadata_ptr_, nullptr);
delete checkpoint;
ImportColumnFamilyOptions import_options; ImportColumnFamilyOptions import_options;
import_options.move_files = false; import_options.move_files = false;
@ -407,6 +408,7 @@ TEST_F(ImportColumnFamilyTest, ImportExportedSSTFromAnotherDB) {
ASSERT_OK(checkpoint->ExportColumnFamily(handles_[1], export_files_dir_, ASSERT_OK(checkpoint->ExportColumnFamily(handles_[1], export_files_dir_,
&metadata_ptr_)); &metadata_ptr_));
ASSERT_NE(metadata_ptr_, nullptr); ASSERT_NE(metadata_ptr_, nullptr);
delete checkpoint;
// Create a new db and import the files. // Create a new db and import the files.
DB* db_copy; DB* db_copy;
@ -424,6 +426,7 @@ TEST_F(ImportColumnFamilyTest, ImportExportedSSTFromAnotherDB) {
ASSERT_EQ(Get(1, Key(i)), value); ASSERT_EQ(Get(1, Key(i)), value);
} }
db_copy->DropColumnFamily(cfh); db_copy->DropColumnFamily(cfh);
db_copy->DestroyColumnFamilyHandle(cfh);
test::DestroyDir(env_, dbname_ + "/db_copy"); test::DestroyDir(env_, dbname_ + "/db_copy");
} }