From 6c5ef3a088e78f1bef93b0f4b2df8e16aaabe987 Mon Sep 17 00:00:00 2001 From: Josh Lind Date: Tue, 1 Oct 2024 21:56:28 -0400 Subject: [PATCH] [State Sync] Remove clone() calls for storage synchronizer. --- state-sync/state-sync-driver/src/bootstrapper.rs | 8 ++++---- state-sync/state-sync-driver/src/continuous_syncer.rs | 8 ++++---- state-sync/state-sync-driver/src/utils.rs | 8 ++++---- 3 files changed, 12 insertions(+), 12 deletions(-) diff --git a/state-sync/state-sync-driver/src/bootstrapper.rs b/state-sync/state-sync-driver/src/bootstrapper.rs index 3299577453085..296b75d6ca78d 100644 --- a/state-sync/state-sync-driver/src/bootstrapper.rs +++ b/state-sync/state-sync-driver/src/bootstrapper.rs @@ -1177,7 +1177,7 @@ impl< BootstrappingMode::ApplyTransactionOutputsFromGenesis => { if let Some(transaction_outputs_with_proof) = transaction_outputs_with_proof { utils::apply_transaction_outputs( - self.storage_synchronizer.clone(), + &mut self.storage_synchronizer, notification_metadata, proof_ledger_info, end_of_epoch_ledger_info, @@ -1198,7 +1198,7 @@ impl< BootstrappingMode::ExecuteTransactionsFromGenesis => { if let Some(transaction_list_with_proof) = transaction_list_with_proof { utils::execute_transactions( - self.storage_synchronizer.clone(), + &mut self.storage_synchronizer, notification_metadata, proof_ledger_info, end_of_epoch_ledger_info, @@ -1219,7 +1219,7 @@ impl< BootstrappingMode::ExecuteOrApplyFromGenesis => { if let Some(transaction_list_with_proof) = transaction_list_with_proof { utils::execute_transactions( - self.storage_synchronizer.clone(), + &mut self.storage_synchronizer, notification_metadata, proof_ledger_info, end_of_epoch_ledger_info, @@ -1229,7 +1229,7 @@ impl< } else if let Some(transaction_outputs_with_proof) = transaction_outputs_with_proof { utils::apply_transaction_outputs( - self.storage_synchronizer.clone(), + &mut self.storage_synchronizer, notification_metadata, proof_ledger_info, end_of_epoch_ledger_info, diff --git a/state-sync/state-sync-driver/src/continuous_syncer.rs b/state-sync/state-sync-driver/src/continuous_syncer.rs index c8fc254c976a2..d8da9a4496d52 100644 --- a/state-sync/state-sync-driver/src/continuous_syncer.rs +++ b/state-sync/state-sync-driver/src/continuous_syncer.rs @@ -302,7 +302,7 @@ impl< ContinuousSyncingMode::ApplyTransactionOutputs => { if let Some(transaction_outputs_with_proof) = transaction_outputs_with_proof { utils::apply_transaction_outputs( - self.storage_synchronizer.clone(), + &mut self.storage_synchronizer, notification_metadata, ledger_info_with_signatures.clone(), None, @@ -323,7 +323,7 @@ impl< ContinuousSyncingMode::ExecuteTransactions => { if let Some(transaction_list_with_proof) = transaction_list_with_proof { utils::execute_transactions( - self.storage_synchronizer.clone(), + &mut self.storage_synchronizer, notification_metadata, ledger_info_with_signatures.clone(), None, @@ -344,7 +344,7 @@ impl< ContinuousSyncingMode::ExecuteTransactionsOrApplyOutputs => { if let Some(transaction_list_with_proof) = transaction_list_with_proof { utils::execute_transactions( - self.storage_synchronizer.clone(), + &mut self.storage_synchronizer, notification_metadata, ledger_info_with_signatures.clone(), None, @@ -354,7 +354,7 @@ impl< } else if let Some(transaction_outputs_with_proof) = transaction_outputs_with_proof { utils::apply_transaction_outputs( - self.storage_synchronizer.clone(), + &mut self.storage_synchronizer, notification_metadata, ledger_info_with_signatures.clone(), None, diff --git a/state-sync/state-sync-driver/src/utils.rs b/state-sync/state-sync-driver/src/utils.rs index 25104ad0fda7c..24ca021cca373 100644 --- a/state-sync/state-sync-driver/src/utils.rs +++ b/state-sync/state-sync-driver/src/utils.rs @@ -376,8 +376,8 @@ pub fn update_new_epoch_metrics() { /// Executes the given list of transactions and /// returns the number of transactions in the list. -pub async fn execute_transactions( - mut storage_synchronizer: StorageSyncer, +pub async fn execute_transactions( + storage_synchronizer: &mut StorageSyncer, notification_metadata: NotificationMetadata, proof_ledger_info: LedgerInfoWithSignatures, end_of_epoch_ledger_info: Option, @@ -397,8 +397,8 @@ pub async fn execute_transactions( - mut storage_synchronizer: StorageSyncer, +pub async fn apply_transaction_outputs( + storage_synchronizer: &mut StorageSyncer, notification_metadata: NotificationMetadata, proof_ledger_info: LedgerInfoWithSignatures, end_of_epoch_ledger_info: Option,