Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

remove expected rent collection and rehashing completely #28422

Merged
merged 1 commit into from
Oct 17, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
2 changes: 0 additions & 2 deletions accounts-bench/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,6 @@ fn main() {
&ancestors,
&EpochSchedule::default(),
&RentCollector::default(),
true,
);
time.stop();
let mut time_store = Measure::start("hash using store");
Expand All @@ -137,7 +136,6 @@ fn main() {
None,
&EpochSchedule::default(),
&RentCollector::default(),
false,
true,
);
time_store.stop();
Expand Down
4 changes: 0 additions & 4 deletions core/src/accounts_hash_verifier.rs
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,6 @@ impl AccountsHashVerifier {
rent_collector: &accounts_package.rent_collector,
store_detailed_debug_info_on_failure: false,
full_snapshot: None,
enable_rehashing: accounts_package.enable_rehashing,
},
&sorted_storages,
timings,
Expand All @@ -255,7 +254,6 @@ impl AccountsHashVerifier {
rent_collector: &accounts_package.rent_collector,
store_detailed_debug_info_on_failure: false,
full_snapshot: None,
enable_rehashing: accounts_package.enable_rehashing,
},
);
info!(
Expand All @@ -275,7 +273,6 @@ impl AccountsHashVerifier {
// now that we've failed, store off the failing contents that produced a bad capitalization
store_detailed_debug_info_on_failure: true,
full_snapshot: None,
enable_rehashing: accounts_package.enable_rehashing,
},
&sorted_storages,
HashStats::default(),
Expand Down Expand Up @@ -564,7 +561,6 @@ mod tests {
accounts: Arc::clone(&accounts),
epoch_schedule: EpochSchedule::default(),
rent_collector: RentCollector::default(),
enable_rehashing: true,
};

AccountsHashVerifier::process_accounts_package(
Expand Down
1 change: 0 additions & 1 deletion core/tests/epoch_accounts_hash.rs
Original file line number Diff line number Diff line change
Expand Up @@ -320,7 +320,6 @@ fn test_epoch_accounts_hash_basic(test_environment: TestEnvironment) {
rent_collector: bank.rent_collector(),
store_detailed_debug_info_on_failure: false,
full_snapshot: None,
enable_rehashing: true,
},
)
.unwrap();
Expand Down
3 changes: 0 additions & 3 deletions runtime/benches/accounts.rs
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,6 @@ fn test_accounts_hash_bank_hash(bencher: &mut Bencher) {
&ancestors,
&EpochSchedule::default(),
&RentCollector::default(),
true,
);
let test_hash_calculation = false;
bencher.iter(|| {
Expand All @@ -112,7 +111,6 @@ fn test_accounts_hash_bank_hash(bencher: &mut Bencher) {
&RentCollector::default(),
false,
false,
true,
false,
))
});
Expand All @@ -137,7 +135,6 @@ fn test_update_accounts_hash(bencher: &mut Bencher) {
&ancestors,
&EpochSchedule::default(),
&RentCollector::default(),
true,
);
});
}
Expand Down
4 changes: 0 additions & 4 deletions runtime/src/accounts.rs
Original file line number Diff line number Diff line change
Expand Up @@ -787,7 +787,6 @@ impl Accounts {
debug_verify: bool,
epoch_schedule: &EpochSchedule,
rent_collector: &RentCollector,
enable_rehashing: bool,
) -> u64 {
let use_index = false;
let is_startup = true;
Expand All @@ -804,7 +803,6 @@ impl Accounts {
epoch_schedule,
rent_collector,
is_startup,
enable_rehashing,
)
.1
}
Expand All @@ -822,7 +820,6 @@ impl Accounts {
rent_collector: &RentCollector,
ignore_mismatch: bool,
store_detailed_debug_info: bool,
enable_rehashing: bool,
use_bg_thread_pool: bool,
) -> bool {
if let Err(err) = self.accounts_db.verify_bank_hash_and_lamports_new(
Expand All @@ -834,7 +831,6 @@ impl Accounts {
rent_collector,
ignore_mismatch,
store_detailed_debug_info,
enable_rehashing,
use_bg_thread_pool,
) {
warn!("verify_bank_hash failed: {:?}, slot: {}", err, slot);
Expand Down
2 changes: 0 additions & 2 deletions runtime/src/accounts_background_service.rs
Original file line number Diff line number Diff line change
Expand Up @@ -340,8 +340,6 @@ impl SnapshotRequestHandler {
rent_collector: snapshot_root_bank.rent_collector(),
store_detailed_debug_info_on_failure: false,
full_snapshot: None,
enable_rehashing: snapshot_root_bank
.bank_enable_rehashing_on_accounts_hash(),
},
)
.unwrap();
Expand Down
Loading