starknet_patricia_storage: add config field to rocksDB

[nimrod-starkware]

Note

Low Risk
Config-only plumbing with default behavior unchanged (false), and no functional read-path logic changes in this PR.

Overview
Adds a new RocksDB tuning parameter, spawn_blocking_reads, to RocksDbStorageConfig (including defaults and config serialization) and exposes it via the node config schema.

Updates deployment committer config templates (committer_config.json and replacer_committer_config.json) to include the new committer_config.storage_config.inner_storage_config.spawn_blocking_reads field (default false).

^{Written by Cursor Bugbot for commit de721d6. 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 )
• 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 👈 (View in Graphite)
• 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.

[chatgpt-codex-connector]

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

[cursor]

Config field spawn_blocking_reads is never used

Low Severity

The new spawn_blocking_reads config field is added to RocksDbStorageConfig but is never read to control any runtime behavior. The get and mget methods unconditionally call spawn_blocking regardless of this setting. Additionally, the config isn't stored in RocksDbStorage (only RocksDbOptions is stored), so the field wouldn't be accessible even if conditional logic were added. Existing TODO comments on lines 291 and 302 explicitly note that the config should control whether to spawn blocking tasks, but this implementation doesn't connect the new field to that behavior.

 

[ArielElp]

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