-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Open
Labels
A-tx-poolRelated to the transaction mempoolRelated to the transaction mempoolC-perfA change motivated by improving speed, memory usage or disk footprintA change motivated by improving speed, memory usage or disk footprint
Description
Describe the feature
When many network transactions are imported at once, this snippet becomes a bottleneck
reth/crates/net/network/src/transactions/mod.rs
Line 1376 in b7d8815
| let tx = match tx.try_into_recovered() { |
This is problematic for two reasons:
- Senders are recovered sequentially.
- This is done in an async task, and with many import tasks happening simultaneously, can block the runtime for other async tasks.
We should instead recover the senders in parallel in a separate blocking task.
Additional context
No response
Metadata
Metadata
Assignees
Labels
A-tx-poolRelated to the transaction mempoolRelated to the transaction mempoolC-perfA change motivated by improving speed, memory usage or disk footprintA change motivated by improving speed, memory usage or disk footprint
Type
Projects
Status
Backlog