-
Notifications
You must be signed in to change notification settings - Fork 618
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[resharding] Include state_changes in ApplySplitStateResult for split_state #9420
Comments
Thanks for the nice description! Makes sense. I couldn't find the reference to |
@wacban, right, so, this task is to populate Specifically, we use it here as
|
For more context please view #9418
During resharding we need to split the trie/state changes across the child shards. This is mainly handled by the
tries.apply_state_changes_to_split_states
function which takes the set of trie state updates and distributes it as trie updates across the child shards as per account id.During pre-processing
runtime.apply_update_to_split_states
function callstries.apply_state_changes_to_split_states
and saves the state changes asApplySplitStateResult
which would later be applied to the child shards during postprocessing after catchup.Note that the
WrappedTrieChanges
(for each child shard) also contains the field calledstate_changes
which wasn't being populated. This would eventually be used in theprocess_split_state
function where we apply and commit the changes to the child shards to DB. We would use the state changes to update flat storage.The text was updated successfully, but these errors were encountered: