Clean up: banking_stage_prepare_sanitized_batch (#28841)

Use measure! for bank.prepare_sanitized_batch_with_results
This commit is contained in:
apfitzge 2022-11-18 14:04:44 -06:00 committed by GitHub
parent c32377b5af
commit a636038fff
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 6 deletions

View File

@ -1507,9 +1507,9 @@ impl BankingStage {
// Only lock accounts for those transactions are selected for the block;
// Once accounts are locked, other threads cannot encode transactions that will modify the
// same account state
let mut lock_time = Measure::start("lock_time");
let batch = bank.prepare_sanitized_batch_with_results(txs, transactions_qos_results.iter());
lock_time.stop();
let (batch, lock_time) = measure!(
bank.prepare_sanitized_batch_with_results(txs, transactions_qos_results.iter())
);
// retryable_txs includes AccountInUse, WouldExceedMaxBlockCostLimit
// WouldExceedMaxAccountCostLimit, WouldExceedMaxVoteCostLimit
@ -1524,10 +1524,8 @@ impl BankingStage {
log_messages_bytes_limit,
);
let mut unlock_time = Measure::start("unlock_time");
// Once the accounts are new transactions can enter the pipeline to process them
drop(batch);
unlock_time.stop();
let (_, unlock_time) = measure!(drop(batch));
let ExecuteAndCommitTransactionsOutput {
ref mut retryable_transaction_indexes,