bug(engine): persisted executed block retrieval

[rkrasiuk]

Description

Currently, new tree constructs executed persisted blocks by calling provider.get_state

reth/crates/engine/tree/src/tree/mod.rs

Lines 1496 to 1504 in 5f9c43c

	let SealedBlockWithSenders { block, senders } = self
	.provider
	.sealed_block_with_senders(hash.into(), TransactionVariant::WithHash)?
	.ok_or_else(|| ProviderError::HeaderNotFound(hash.into()))?;
	let execution_output = self
	.provider
	.get_state(block.number)?
	.ok_or_else(|| ProviderError::StateForNumberNotFound(block.number))?;
	let hashed_state = execution_output.hash_state_slow();

However, this function can only retrieve the state for blocks at tip as it only looks at plain state tables

reth/crates/storage/provider/src/providers/database/provider.rs

Lines 827 to 832 in 5f9c43c

	// This is not working for blocks that are not at tip. as plain state is not the last
	// state of end range. We should rename the functions or add support to access
	// History state. Accessing history state can be tricky but we are not gaining
	// anything.
	let mut plain_accounts_cursor = self.tx.cursor_read::<tables::PlainAccountState>()?;
	let mut plain_storage_cursor = self.tx.cursor_dup_read::<tables::PlainStorageState>()?;