Don't restrict database error variant on executor

[mattsse]

Describe the feature

This should not require ProviderError and should just require Error or Into<BlockExecutionError>

reth/crates/evm/src/execute.rs

Line 123 in ead753d

type Executor<DB: Database<Error = ProviderError>>: for<'a> Executor<

blocked by #8088

Additional context

No response

[github-actions]

This issue is stale because it has been open for 21 days with no activity.

[sergerad]

@mattsse I have looked into this one out of interest. Wanted to ask if you thought the diffs in the PR are on the right track or not.

I assume I can get around the current compiler error by implementing the appropriate from() or into(). But it feels like that could be going down the wrong track and likely to hit many more issues after that.

required for `revm_primitives::EVMError<std::boxed::Box<dyn std::error::Error>>` to implement `std::convert::Into<std::boxed::Box<revm_primitives::EVMError<reth_evm::execute::ProviderError>>>`

Any insight / direction would be appreciated, thanks 🙏

[mattsse]

ah sorry I forgot to link the relevant issue #8152

this problem will go away with an upcoming stable rust feature, on nightly this already works