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

Jump to bottom

perf: prevent changed senders clone #13194

Closed
wants to merge 1 commit into from
Closed

perf: prevent changed senders clone #13194

wants to merge 1 commit into from

Conversation

mattsse
Copy link
Collaborator

@mattsse mattsse commented Dec 6, 2024

ref #13187 #13193

cc @hai-rise

prevents a clone by changing the order, this is okay because all_transactions is entirely separate from sender_info

@mattsse mattsse added C-perf A change motivated by improving speed, memory usage or disk footprint A-tx-pool Related to the transaction mempool labels Dec 6, 2024
@mattsse mattsse requested a review from Rjected December 6, 2024 20:47
@mattsse mattsse mentioned this pull request Dec 6, 2024
Merged
@mattsse
Copy link
Collaborator Author

mattsse commented Dec 6, 2024

dup #13189

@mattsse mattsse closed this Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Rjected Rjected Awaiting requested review from Rjected

Assignees
No one assigned
Labels
A-tx-pool Related to the transaction mempool C-perf A change motivated by improving speed, memory usage or disk footprint
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mattsse