Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions crates/apollo_batcher/src/batcher.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1033,8 +1033,6 @@ impl Batcher {
.compiled_class_hashes_for_migration,
parent_proposal_commitment,
#[cfg(feature = "os_input")]
accessed_keys,
#[cfg(feature = "os_input")]
initial_reads,
},
})
Expand Down
4 changes: 0 additions & 4 deletions crates/apollo_batcher_types/src/batcher_types.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@ use std::ops::Deref;

use blockifier::blockifier::transaction_executor::CompiledClassHashesForMigration;
use blockifier::bouncer::{BouncerWeights, CasmHashComputationData};
#[cfg(feature = "os_input")]
use blockifier::state::accessed_keys::AccessedKeys;
use blockifier::state::cached_state::CommitmentStateDiff;
#[cfg(feature = "os_input")]
use blockifier::state::cached_state::StateMaps;
Expand Down Expand Up @@ -162,8 +160,6 @@ pub struct CentralObjects {
pub casm_hash_computation_data_proving_gas: CasmHashComputationData,
pub compiled_class_hashes_for_migration: CompiledClassHashesForMigration,
pub parent_proposal_commitment: Option<ProposalCommitment>,
#[cfg(feature = "os_input")]
pub accessed_keys: AccessedKeys,
/// Pre-block read values the OS needs to replay the block.
#[cfg(feature = "os_input")]
pub initial_reads: StateMaps,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,6 @@ use blockifier::fee::resources::{
TransactionResources,
};
#[cfg(feature = "os_input")]
use blockifier::state::accessed_keys::AccessedKeys;
#[cfg(feature = "os_input")]
use blockifier::state::cached_state::StateMaps;
use blockifier::state::cached_state::{
CommitmentStateDiff,
Expand Down Expand Up @@ -786,8 +784,6 @@ fn central_blob() -> AerospikeBlob {
#[cfg(feature = "os_input")]
recent_state_commitment_infos: recent_state_commitment_infos(),
#[cfg(feature = "os_input")]
accessed_keys: AccessedKeys::default(),
#[cfg(feature = "os_input")]
initial_reads: StateMaps::default(),
};

Expand Down Expand Up @@ -821,8 +817,6 @@ fn central_blob_with_empty_or_none_fields() -> AerospikeBlob {
#[cfg(feature = "os_input")]
recent_state_commitment_infos: vec![],
#[cfg(feature = "os_input")]
accessed_keys: AccessedKeys::default(),
#[cfg(feature = "os_input")]
initial_reads: StateMaps::default(),
};

Expand Down Expand Up @@ -1179,15 +1173,14 @@ fn serialize_central_objects(#[case] rust_obj: impl Serialize, #[case] python_js
let python_json: serde_json::Value = read_json_file(python_json_path);
let rust_json = serde_json::to_value(rust_obj).unwrap();

// `recent_state_commitment_infos`, `accessed_keys` and `initial_reads` are os_input-only and
// absent from the central (python) blob, so strip them before comparing.
// `recent_state_commitment_infos` and `initial_reads` are os_input-only and absent from the
// central (python) blob, so strip them before comparing.
// TODO(Itamar): Remove this stripping once the python blob includes the fields.
#[cfg(feature = "os_input")]
let rust_json = {
let mut rust_json = rust_json;
if let Some(object) = rust_json.as_object_mut() {
object.remove("recent_state_commitment_infos");
object.remove("accessed_keys");
object.remove("initial_reads");
}
rust_json
Expand Down
8 changes: 0 additions & 8 deletions crates/apollo_consensus_orchestrator/src/cende/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,6 @@ use async_trait::async_trait;
use blockifier::abi::constants::STORED_BLOCK_HASH_BUFFER;
use blockifier::blockifier::transaction_executor::CompiledClassHashesForMigration;
use blockifier::bouncer::{BouncerWeights, CasmHashComputationData};
#[cfg(feature = "os_input")]
use blockifier::state::accessed_keys::AccessedKeys;
use blockifier::state::cached_state::CommitmentStateDiff;
#[cfg(feature = "os_input")]
use blockifier::state::cached_state::StateMaps;
Expand Down Expand Up @@ -109,8 +107,6 @@ pub struct AerospikeBlob {
#[cfg(feature = "os_input")]
recent_state_commitment_infos: Vec<StateCommitmentInfosAndNumber>,
#[cfg(feature = "os_input")]
accessed_keys: AccessedKeys,
#[cfg(feature = "os_input")]
initial_reads: StateMaps,
}

Expand Down Expand Up @@ -408,8 +404,6 @@ pub struct BlobParameters {
#[cfg(feature = "os_input")]
pub recent_state_commitment_infos: Vec<StateCommitmentInfosAndNumber>,
#[cfg(feature = "os_input")]
pub accessed_keys: AccessedKeys,
#[cfg(feature = "os_input")]
pub initial_reads: StateMaps,
}

Expand Down Expand Up @@ -469,8 +463,6 @@ impl AerospikeBlob {
#[cfg(feature = "os_input")]
recent_state_commitment_infos: blob_parameters.recent_state_commitment_infos,
#[cfg(feature = "os_input")]
accessed_keys: blob_parameters.accessed_keys,
#[cfg(feature = "os_input")]
initial_reads: blob_parameters.initial_reads,
})
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -621,8 +621,6 @@ impl SequencerConsensusContext {
.collect_recent_state_commitment_infos(height)
.await,
#[cfg(feature = "os_input")]
accessed_keys: central_objects.accessed_keys,
#[cfg(feature = "os_input")]
initial_reads: central_objects.initial_reads,
})
.await
Expand Down
Loading