Skip to content

Commit

Permalink
secondary instance: remove unnessisary cfds_changed->count() (#13086)
Browse files Browse the repository at this point in the history
Summary:
`cfds_changed->count(cfd)` is not needed, just blind insert.

Pull Request resolved: facebook/rocksdb#13086

Reviewed By: hx235

Differential Revision: D64712400

Pulled By: cbi42

fbshipit-source-id: 4ef62aaa724c8397baa4ff350c16a7a8d04d7067
  • Loading branch information
rockeet authored and facebook-github-bot committed Oct 29, 2024
1 parent ddafba8 commit 8109046
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions db/db_impl/db_impl_secondary.cc
Original file line number Diff line number Diff line change
Expand Up @@ -247,9 +247,7 @@ Status DBImplSecondary::RecoverLogFiles(
if (cfd == nullptr) {
continue;
}
if (cfds_changed->count(cfd) == 0) {
cfds_changed->insert(cfd);
}
cfds_changed->insert(cfd);
const std::vector<FileMetaData*>& l0_files =
cfd->current()->storage_info()->LevelFiles(0);
SequenceNumber seq =
Expand Down

0 comments on commit 8109046

Please sign in to comment.