Skip to content

starknet_patricia_storage: rename cached storage#12734

Open
nimrod-starkware wants to merge 1 commit intonimrod/parallel-reads/commiter-config-validationfrom
nimrod/parallel-reads/rename-cached-storage
Open

starknet_patricia_storage: rename cached storage#12734
nimrod-starkware wants to merge 1 commit intonimrod/parallel-reads/commiter-config-validationfrom
nimrod/parallel-reads/rename-cached-storage

Conversation

@nimrod-starkware
Copy link
Contributor

@nimrod-starkware nimrod-starkware commented Feb 19, 2026

Note

Low Risk
Primarily a type/name migration across crates; behavior should be unchanged, with risk limited to compilation/integration fallout from missed references.

Overview
Switches the committer and CLI benchmark “cached storage” wrapper from CachedStorage to the newly named PatriciaCachedStorage across Apollo committer construction and config typing.

Renames the implementation in starknet_patricia_storage to PatriciaCachedStorage (keeping CachedStorageConfig/stats naming) and updates tests/imports to match, without changing the caching behavior in the wrapper.

Written by Cursor Bugbot for commit 53923c7. This will update automatically on new commits. Configure here.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Contributor Author

nimrod-starkware commented Feb 19, 2026

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

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

@nimrod-starkware nimrod-starkware force-pushed the nimrod/parallel-reads/commiter-config-validation branch from d0ff4f3 to 8a11f44 Compare February 19, 2026 14:12
@nimrod-starkware nimrod-starkware force-pushed the nimrod/parallel-reads/rename-cached-storage branch from c104a8b to 53923c7 Compare February 19, 2026 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments