Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Uses IntSet in AccountsDb::calc_delete_dependencies() #33528

Merged
merged 1 commit into from
Oct 4, 2023
Merged

Uses IntSet in AccountsDb::calc_delete_dependencies() #33528

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions accounts-db/src/accounts_db.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2816,7 +2816,7 @@ impl AccountsDb {
// Another pass to check if there are some filtered accounts which
// do not match the criteria of deleting all appendvecs which contain them
// then increment their storage count.
let mut already_counted = HashSet::new();
let mut already_counted = IntSet::default();
for (pubkey, (account_infos, ref_count_from_storage)) in purges.iter() {
let mut failed_slot = None;
let all_stores_being_deleted =
Expand Down Expand Up @@ -2861,7 +2861,7 @@ impl AccountsDb {
}

// increment store_counts to non-zero for all stores that can not be deleted.
let mut pending_stores = HashSet::new();
let mut pending_stores = IntSet::default();
for (slot, _account_info) in account_infos {
if !already_counted.contains(slot) {
pending_stores.insert(*slot);
Expand Down