Bugfix/request receipts

[flcl42]

Fixes #5904

Changes

• Load receipts in case of fast sync transition
• Set state block number to recently processed block number

Types of changes

What types of changes does your code introduce?

• Bugfix (a non-breaking change that fixes an issue)
• New feature (a non-breaking change that adds functionality)
• Breaking change (a change that causes existing functionality not to work as expected)
• Optimization
• Refactoring
• Documentation update
• Build-related changes
• Other: Description

Testing

Requires testing

• Yes
• No

If yes, did you write tests?

• Yes
• No

[benaadams]

Hive tests https://github.com/NethermindEth/nethermind/actions/runs/7835544801/job/21381168735

[benaadams]

Resolves Hive

Sync after 2 blocks - Withdrawals on Block 2 - Multiple Withdrawal Accounts (Paris) (nethermind)

This one still outstanding though

Sync after 128 blocks - Withdrawals on Block 2 - Multiple Withdrawal Accounts (Paris) (nethermind)

[LukaszRozmej]

Can you explain logic behind this changes?

[asdacap]

Just an idera, why not make fast and full use the same downloader so that we don't have to guess which part start from which block at which thread.

[flcl42]

Just an idera, why not make fast and full use the same downloader so that we don't have to guess which part start from which block at which thread.

So you need to involve state sync too I guess, because it defines the moment when we switch from fast sync(bodies and receipts) to full sync(bodies and processing).

[LukaszRozmej]

Can you explain logic behind this changes?

?

[smartprogrammer93]

@kamilchodola run smoke tests on this PR before merging

[marcindsobczak]

Sync after 2 blocks - Withdrawals on Block 2 - Multiple Withdrawal Accounts (nethermind) 🟢 fixed
Sync after 128 blocks - Withdrawals on Block 2 - Multiple Withdrawal Accounts (nethermind) 🔴 still failing

https://github.com/NethermindEth/nethermind/actions/runs/8156727376/job/22294826941