Skip to content

feat(pruner): transaction senders#3912

Merged
shekhirin merged 13 commits intomainfrom
alexey/pruner-transaction-senders
Aug 1, 2023
Merged

feat(pruner): transaction senders#3912
shekhirin merged 13 commits intomainfrom
alexey/pruner-transaction-senders

Conversation

@shekhirin
Copy link
Member

@shekhirin shekhirin commented Jul 25, 2023

Resolves #3695

Very similar to #3892

2023-07-26T18:38:37.150193Z TRACE prune_receipts{to_block=3969468 prune_mode=Distance(64)}: pruner: Pruned receipts entries=74 progress="100.0%"
2023-07-26T18:38:37.156114Z TRACE prune_transaction_lookup{to_block=3969532 prune_mode=Full}: pruner: Pruned transaction lookup entries=39 progress="100.0%"
2023-07-26T18:38:37.156573Z TRACE prune_transaction_senders{to_block=3969468 prune_mode=Distance(64)}: pruner: Pruned transaction senders entries=74 progress="100.0%"
Table Name # Entries Branch Pages Leaf Pages Overflow Pages Total Size
PruneCheckpoints 3 0 1 0 4 KiB
Receipts 3167 3 105 0 432 KiB
TxHashNumber 0 0 0 0 0 B
TxSenders 3167 1 31 0 128 KiB

@shekhirin shekhirin force-pushed the alexey/pruner-transaction-senders branch 2 times, most recently from 6962f44 to b8a4ddc Compare July 25, 2023 16:39
@shekhirin shekhirin force-pushed the alexey/pruner-transaction-lookup branch from 4e42e78 to 6ba3ea4 Compare July 25, 2023 16:40
@shekhirin shekhirin marked this pull request as draft July 25, 2023 16:43
@shekhirin shekhirin force-pushed the alexey/pruner-transaction-senders branch 2 times, most recently from a7e6332 to 33d87f4 Compare July 25, 2023 16:52
@shekhirin shekhirin force-pushed the alexey/pruner-transaction-lookup branch from 6ba3ea4 to 7228862 Compare July 25, 2023 16:52
@shekhirin shekhirin added C-enhancement New feature or request A-pruning Related to pruning or full node labels Jul 25, 2023
Base automatically changed from alexey/pruner-transaction-lookup to main July 25, 2023 17:49
@shekhirin shekhirin force-pushed the alexey/pruner-transaction-senders branch from e23b932 to afbb975 Compare July 25, 2023 18:00
@codecov
Copy link

codecov bot commented Jul 26, 2023

Codecov Report

Merging #3912 (668b482) into main (dee14c7) will increase coverage by 0.02%.
Report is 1 commits behind head on main.
The diff coverage is 93.63%.

Impacted file tree graph

Files Changed Coverage Δ
crates/primitives/src/prune/target.rs 80.95% <ø> (ø)
crates/prune/src/pruner.rs 88.73% <88.09%> (-0.26%) ⬇️
crates/stages/src/stages/sender_recovery.rs 92.45% <100.00%> (+1.51%) ⬆️
crates/stages/src/test_utils/test_db.rs 84.16% <100.00%> (+0.70%) ⬆️

... and 14 files with indirect coverage changes

Flag Coverage Δ
integration-tests 16.20% <0.00%> (-0.06%) ⬇️
unit-tests 64.22% <93.63%> (+0.03%) ⬆️

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

Components Coverage Δ
reth binary 25.63% <ø> (-0.07%) ⬇️
blockchain tree 83.04% <ø> (ø)
pipeline 90.14% <100.00%> (+0.11%) ⬆️
storage (db) 74.30% <ø> (ø)
trie 94.70% <ø> (ø)
txpool 46.00% <ø> (ø)
networking 77.45% <ø> (-0.06%) ⬇️
rpc 58.36% <ø> (ø)
consensus 63.51% <ø> (ø)
revm 33.10% <ø> (ø)
payload builder 6.58% <ø> (ø)
primitives 88.06% <ø> (+0.01%) ⬆️

@shekhirin shekhirin marked this pull request as ready for review July 26, 2023 18:38
@shekhirin shekhirin added this pull request to the merge queue Aug 1, 2023
Merged via the queue into main with commit 4688fd2 Aug 1, 2023
@shekhirin shekhirin deleted the alexey/pruner-transaction-senders branch August 1, 2023 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-pruning Related to pruning or full node C-enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Respect Sender Recovery prune part in the pruner

3 participants