Skip to content

perf: drop lock early#5016

Merged
mattsse merged 1 commit intomainfrom
matt/drop-lock-early
Oct 13, 2023
Merged

perf: drop lock early#5016
mattsse merged 1 commit intomainfrom
matt/drop-lock-early

Conversation

@mattsse
Copy link
Collaborator

@mattsse mattsse commented Oct 13, 2023

drop lock earlier

@mattsse mattsse added the A-tx-pool Related to the transaction mempool label Oct 13, 2023
@codecov
Copy link

codecov bot commented Oct 13, 2023

Codecov Report

Merging #5016 (18a9ff8) into main (92ceb2f) will decrease coverage by 0.04%.
Report is 1 commits behind head on main.
The diff coverage is 0.00%.

Impacted file tree graph

Files Coverage Δ
crates/transaction-pool/src/validate/task.rs 3.26% <0.00%> (ø)

... and 13 files with indirect coverage changes

Flag Coverage Δ
integration-tests 15.84% <0.00%> (+<0.01%) ⬆️
unit-tests 62.29% <0.00%> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
reth binary 30.85% <ø> (ø)
blockchain tree 80.82% <ø> (ø)
pipeline 88.37% <ø> (ø)
storage (db) 74.40% <ø> (ø)
trie 94.48% <ø> (-0.04%) ⬇️
txpool 48.05% <0.00%> (-0.47%) ⬇️
networking 76.45% <ø> (-0.02%) ⬇️
rpc 57.92% <ø> (+0.02%) ⬆️
consensus 63.01% <ø> (ø)
revm 27.89% <ø> (ø)
payload builder 7.96% <ø> (ø)
primitives 86.00% <ø> (-0.04%) ⬇️

@mattsse mattsse merged commit 920e3f0 into main Oct 13, 2023
@mattsse mattsse deleted the matt/drop-lock-early branch October 13, 2023 20:24
mattsse added a commit that referenced this pull request Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-tx-pool Related to the transaction mempool

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant