starknet_patricia_storage: flush explictly to cache

[nimrod-starkware]

Note

Medium Risk
Introduces a new mutable cache update path that bulk-inserts entries and can change runtime caching behavior; it also uses Arc::get_mut(...).expect(...), so incorrect sharing assumptions could cause panics.

Overview
CachedStorage gains a new flush_to_cache(map: DbHashMap) method that explicitly bulk-populates the LRU cache with a set of key/value pairs, bypassing the existing per-write caching behavior.

This enables callers to proactively warm/refresh the cache from a batch map rather than relying on set/mset updates only.

^{Written by Cursor Bugbot for commit cc406a4. This will update automatically on new commits. Configure here.}

[reviewable-StarkWare]

This change is 

[nimrod-starkware]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• starknet_patricia_storage: flush explictly to cache #12267 : 2 dependent PRs (#12268 , #12269 ) 👈 (View in Graphite)
• starknet_committer: create storage tries based on config #12441
• starknet_committer: add a field indicating if storage tries should be created concurrently #12440
• starknet_committer: remove code dup for single trie #12201
• starknet_patricia_storage: remove code dup for mget #12175
• starknet_patricia_storage: spawn a task or not based on config #12174
• starknet_patricia_storage: add config field to rocksDB storage #12173
• starknet_patricia_storage: add config field to rocksDB #12172
• starknet_patricia_storage: spawn blocking tasks for rocksDB reads #12168 : 1 other dependent PR (#12169 )
• starknet_committer: parallel reads using unordered futures #12167
• starknet_committer: create sotrage tries sequentially #12166
• starknet_patricia,starknet_committer: dont pass storage mut when not needed #12165
• starknet_patricia_storage: non mut get in storage trait #12123 : 1 other dependent PR (#12252 )
• starknet_patricia_storage: dont cache reads in cached storage #12251
• starknet_patricia_storage: atomic counters for read stats #12122
• main-v0.14.2

This stack of pull requests is managed by Graphite. Learn more about stacking.

[ArielElp]

@ArielElp reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on nimrod-starkware).

---

crates/starknet_patricia_storage/src/map_storage.rs line 227 at r1 (raw file):

    pub fn flush_to_cache(&mut self, map: DbHashMap) {
        let cache =
            Arc::get_mut(&mut self.cache).expect("Failed to get mutable reference to cache.");

We no longer need Arc::get_mut, right?