search

ledgerwatch / erigon

Ethereum implementation on the efficiency frontier
GNU Lesser General Public License v3.0
3.03k stars 1.04k forks source link

re-introduce: `state.NewCachedReader` #10747

Open AskAlexSharov opened 1 week ago

AskAlexSharov commented 1 week ago

we stoped use it here https://github.com/ledgerwatch/erigon/pull/10746 , because of bug (I think reason is: 1. StateCache works on incarnation and 2. Need delete all storage when delete account) https://github.com/ledgerwatch/erigon/issues/10444 But we don't know perf impact of cache. Need carefully re-introduce it and bench speed.

Of-course such cache will be useful for "remote rpcdaemon deploy", but it's rare.

Maybe cache need a bit change to use Domain concept. (not sure)