starknet_committer: add alias for map storage

[nimrod-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[nimrod-starkware]

• starknet_committer: move input impl outside of cli #8338
• starknet_committer: delete duplicated definitions #8322
• starknet_committer: add alias for map storage #8316 👈 (View in Graphite)
• starknet_committer: rename map storage #8315
• starknet_committer: input doesn't take ownership of the storage #8291
• main

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

[dorimedini-starkware]

Reviewed 11 of 11 files at r1, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @nimrod-starkware)

---

crates/starknet_committer_and_os_cli/src/committer_cli/tests/regression_tests.rs line 43 at r1 (raw file):

// TODO(Nimrod): Delete this struct and use `CommitterInputImpl` instead.
struct CommitterInput(Input<ConfigImpl>, MapStorage);

you have a PR for this TODO as well?

Code quote:

// TODO(Nimrod): Delete this struct and use `CommitterInputImpl` instead.
struct CommitterInput(Input<ConfigImpl>, MapStorage);

---

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

use crate::storage_trait::{DbKey, DbValue, Storage};
pub type MapStorage = HashMap<DbKey, DbValue>;
#[derive(Serialize, Debug)]

newlines around type / struct defs

Suggestion:

use crate::storage_trait::{DbKey, DbValue, Storage};

pub type MapStorage = HashMap<DbKey, DbValue>;

#[derive(Serialize, Debug)]

[dorimedini-starkware]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @nimrod-starkware)

[github-actions]

Benchmark movements: No major performance changes detected.

[nimrod-starkware]

Reviewable status: 9 of 11 files reviewed, 2 unresolved discussions (waiting on @dorimedini-starkware)

---

crates/starknet_committer_and_os_cli/src/committer_cli/tests/regression_tests.rs line 43 at r1 (raw file):

Previously, dorimedini-starkware wrote…

you have a PR for this TODO as well?

yes, on top of this:
https://reviewable.io/reviews/starkware-libs/sequencer/8322

---

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

Previously, dorimedini-starkware wrote…

newlines around type / struct defs

Done.

[dorimedini-starkware]

Reviewed 1 of 2 files at r3, 2 of 2 files at r4, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @nimrod-starkware)

[graphite-app]

Merge activity

• Jul 31, 9:25 AM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.