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

Clean up: banking_stage_prepare_sanitized_batch #28841

Conversation

apfitzge
Copy link
Contributor

@apfitzge apfitzge commented Nov 16, 2022

Problem

Explicit Measure::start/stop around very simple blocks.

Summary of Changes

Use measure! so the code is simpler.

Fixes #

@apfitzge apfitzge force-pushed the refactor/banking_stage_prepare_sanitized_batch branch from 2545de4 to cc3a485 Compare November 16, 2022 16:51
@apfitzge apfitzge changed the title Refactor/banking_stage_prepare_sanitized_batch Clean up: banking_stage_prepare_sanitized_batch Nov 16, 2022
@apfitzge apfitzge changed the title Clean up: banking_stage_prepare_sanitized_batch Refactor/banking_stage_prepare_sanitized_batch Nov 16, 2022
@apfitzge apfitzge changed the title Refactor/banking_stage_prepare_sanitized_batch Clean up: banking_stage_prepare_sanitized_batch Nov 16, 2022
@apfitzge apfitzge force-pushed the refactor/banking_stage_prepare_sanitized_batch branch from cc3a485 to 77cfafb Compare November 17, 2022 20:05
@apfitzge apfitzge marked this pull request as ready for review November 17, 2022 20:16
@apfitzge apfitzge requested a review from tao-stones November 17, 2022 20:16
Copy link
Contributor

@tao-stones tao-stones left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@apfitzge apfitzge merged commit a636038 into solana-labs:master Nov 18, 2022
@apfitzge apfitzge deleted the refactor/banking_stage_prepare_sanitized_batch branch November 18, 2022 20:04
gnapoli23 pushed a commit to gnapoli23/solana that referenced this pull request Dec 16, 2022
Use measure! for bank.prepare_sanitized_batch_with_results
nickfrosty pushed a commit to nickfrosty/solana that referenced this pull request Jan 4, 2023
Use measure! for bank.prepare_sanitized_batch_with_results
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants