Moves where the TransactionExecutionDetails::accounts_data_len_delta is interpreted (#34692)

This commit is contained in:
Brooks 2024-01-09 17:39:42 -05:00 committed by GitHub
parent abe699b7b4
commit 642a7e0acb
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 6 deletions

View File

@ -4873,7 +4873,7 @@ impl Bank {
accounts,
return_data,
touched_account_count,
accounts_resize_delta,
accounts_resize_delta: accounts_data_len_delta,
} = transaction_context.into();
if status.is_ok()
@ -4891,7 +4891,6 @@ impl Bank {
loaded_transaction.accounts.len() as u64
);
saturating_add_assign!(timings.details.changed_account_count, touched_account_count);
let accounts_data_len_delta = status.as_ref().map_or(0, |_| accounts_resize_delta);
let return_data = if enable_return_data_recording && !return_data.data.is_empty() {
Some(return_data)
@ -5570,10 +5569,12 @@ impl Bank {
let accounts_data_len_delta = execution_results
.iter()
.filter_map(|execution_result| {
execution_result
.details()
.map(|details| details.accounts_data_len_delta)
.filter_map(TransactionExecutionResult::details)
.filter_map(|details| {
details
.status
.is_ok()
.then_some(details.accounts_data_len_delta)
})
.sum();
self.update_accounts_data_size_delta_on_chain(accounts_data_len_delta);