madara icon indicating copy to clipboard operation
madara copied to clipboard

dev: impl `get_state_updates` using `re_execute_transactions`

Open tdelabro opened this issue 9 months ago • 2 comments

What is the problem?

State updates used to be stored directly onto the disk for each block. They are not anymore since #1580, meaning the RPC won't work anymore.

What to do?

Replace the current implementation with a call to re_execute_transactions adding the option to have the StateDiff. Execute all the tx on the same CachedState, and at the end, return the StateDiff

Remove state_diff and get_state_diff and cairo_job and update_cairo_job from crates/client/db/src/da_db.rs. I think we should keep the last_proved_block logic for now.

tdelabro avatar May 06 '24 14:05 tdelabro

hi! what to try this, if it's not asap

tonypony220 avatar May 06 '24 16:05 tonypony220

@tonypony220 assigned to you! Do you have an ETA in mind?

tdelabro avatar May 06 '24 16:05 tdelabro