dolysis
/
rocksdb
mirror of https://github.com/facebook/rocksdb.git
3
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

secondary instance: remove unnessisary cfds_changed->count() (#13086) 

Summary:
`cfds_changed->count(cfd)` is not needed, just blind insert.

Pull Request resolved: https://github.com/facebook/rocksdb/pull/13086

Reviewed By: hx235

Differential Revision: D64712400

Pulled By: cbi42

fbshipit-source-id: 4ef62aaa724c8397baa4ff350c16a7a8d04d7067
This commit is contained in:
leipeng 2024-10-29 11:04:20 -07:00 committed by Facebook GitHub Bot
parent ddafba870d
commit 8109046222
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
db/db_impl/db_impl_secondary.cc
View File

@ -247,9 +247,7 @@ Status DBImplSecondary::RecoverLogFiles(
if (cfd == nullptr) { if (cfd == nullptr) {
continue; continue;
} }
if (cfds_changed->count(cfd) == 0) {
cfds_changed->insert(cfd); cfds_changed->insert(cfd);
}
const std::vector<FileMetaData*>& l0_files = const std::vector<FileMetaData*>& l0_files =
cfd->current()->storage_info()->LevelFiles(0); cfd->current()->storage_info()->LevelFiles(0);
SequenceNumber seq = SequenceNumber seq =