]> git-server-git.apps.pok.os.sepia.ceph.com Git - rocksdb.git/commit
Fix a bug in file ingestion (#5760)
authorYanqin Jin <yanqin@fb.com>
Sat, 31 Aug 2019 01:27:43 +0000 (18:27 -0700)
committerYanqin Jin <yanqin@fb.com>
Tue, 3 Sep 2019 20:29:55 +0000 (13:29 -0700)
commit5c12a474f30b00b1a14527c194df44ab8cc72deb
tree945998d5a8317fdb4ae6860a2b9b83a87645fd14
parent6d113fc066c5816887eb19c84d12c0677a68af2b
Fix a bug in file ingestion (#5760)

Summary:
Before this PR, when the number of column families involved in a file ingestion exceeds 2, a bug in the looping logic prevents correct file number being assigned to each ingestion job.
Also skip deleting non-existing hard links during cleanup-after-failure.

Test plan (devserver)
```
$COMPILE_WITH_ASAN=1 make all
$./external_sst_file_test --gtest_filter=ExternalSSTFileTest/ExternalSSTFileTest.IngestFilesIntoMultipleColumnFamilies_*/*
$makke check
```
Pull Request resolved: https://github.com/facebook/rocksdb/pull/5760

Differential Revision: D17142982

Pulled By: riversand963

fbshipit-source-id: 06c1847a4e7a402647bcf28d124e70f2a0f9daf6
HISTORY.md
db/db_impl.cc
db/external_sst_file_ingestion_job.cc
db/external_sst_file_test.cc