erigontech / erigon

Ethereum implementation on the efficiency frontier https://erigon.gitbook.io
GNU Lesser General Public License v3.0
3.15k stars 1.13k forks source link

turbo/execution: drop td lookup in getters #12832

Closed taratorio closed 4 days ago

taratorio commented 4 days ago

Since we started prunning TD (roughly after 100,000 blocks) there is no longer any point in keeping the TD lookups in some of the ExecutionModule read APIs such as GetHeader, GetBody, etc.

Those lookups actually also break the APIs for use cases in which some components need to read block data older than the prune point of the TD. In those cases Ive observed we return errors when in reality the data is there (in snapshots) and is accessible otherwise.