starknet_committer: add a field indicating if storage tries should be created concurrently

[nimrod-starkware]

Note

Low Risk
Adds a new boolean config field but defaults it to false everywhere; changes are mostly plumbing and test updates with minimal behavioral impact unless the flag is later enabled.

Overview
Adds a new ReaderConfig flag, build_storage_tries_concurrently, to control whether storage tries are built concurrently vs sequentially, including serialization and a public accessor.

Wires the new setting through config artifacts (committer_config.json, replacer_committer_config.json, and config_schema.json) and updates CLI input parsing/tests to pass the additional ReaderConfig::new parameter (currently hardcoded to false in the CLI with a TODO).

^{Written by Cursor Bugbot for commit 86e286c. 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_committer: create storage tries based on config #12441 : 2 dependent PRs (#12267 , #12269 )
• starknet_committer: add a field indicating if storage tries should be created concurrently #12440 👈 (View in Graphite)
• 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.

[cursor]

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

[cursor]

CLI ignores concurrency reader config

Medium Severity

parse_input hardcodes build_storage_tries_concurrently to false when creating ReaderConfig. Because RawInput/RawConfigImpl has no corresponding field, the CLI path cannot carry this new setting and always runs the sequential behavior, even after adding build_storage_tries_concurrently to committer configuration.

 

[ArielElp]

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