From 1761c0947b3e88024d61c3661661abea27d9fef6 Mon Sep 17 00:00:00 2001 From: Brooks Date: Wed, 12 Apr 2023 12:45:31 -0400 Subject: [PATCH] Removes unused arg from SnapshotGossipManager::new() (#31169) --- core/src/snapshot_packager_service.rs | 11 +---------- .../snapshot_gossip_manager.rs | 6 +----- 2 files changed, 2 insertions(+), 15 deletions(-) diff --git a/core/src/snapshot_packager_service.rs b/core/src/snapshot_packager_service.rs index 4c7f300486..9c0f86c58b 100644 --- a/core/src/snapshot_packager_service.rs +++ b/core/src/snapshot_packager_service.rs @@ -2,9 +2,7 @@ mod snapshot_gossip_manager; use { crossbeam_channel::{Receiver, Sender}, snapshot_gossip_manager::SnapshotGossipManager, - solana_gossip::cluster_info::{ - ClusterInfo, MAX_INCREMENTAL_SNAPSHOT_HASHES, MAX_LEGACY_SNAPSHOT_HASHES, - }, + solana_gossip::cluster_info::{ClusterInfo, MAX_LEGACY_SNAPSHOT_HASHES}, solana_measure::measure_us, solana_perf::thread::renice_this_thread, solana_runtime::{ @@ -49,12 +47,6 @@ impl SnapshotPackagerService { .maximum_full_snapshot_archives_to_retain .get(), ); - let max_incremental_snapshot_hashes = std::cmp::min( - MAX_INCREMENTAL_SNAPSHOT_HASHES, - snapshot_config - .maximum_incremental_snapshot_archives_to_retain - .get(), - ); let t_snapshot_packager = Builder::new() .name("solSnapshotPkgr".to_string()) @@ -65,7 +57,6 @@ impl SnapshotPackagerService { SnapshotGossipManager::new( cluster_info, max_full_snapshot_hashes, - max_incremental_snapshot_hashes, ) ); if let Some(snapshot_gossip_manager) = snapshot_gossip_manager.as_mut() { diff --git a/core/src/snapshot_packager_service/snapshot_gossip_manager.rs b/core/src/snapshot_packager_service/snapshot_gossip_manager.rs index 5e1335ca1a..3fa2c99ad9 100644 --- a/core/src/snapshot_packager_service/snapshot_gossip_manager.rs +++ b/core/src/snapshot_packager_service/snapshot_gossip_manager.rs @@ -21,11 +21,7 @@ pub struct SnapshotGossipManager { impl SnapshotGossipManager { /// Construct a new SnapshotGossipManager with empty snapshot hashes #[must_use] - pub fn new( - cluster_info: Arc, - max_full_snapshot_hashes: usize, - _max_incremental_snapshot_hashes: usize, - ) -> Self { + pub fn new(cluster_info: Arc, max_full_snapshot_hashes: usize) -> Self { SnapshotGossipManager { cluster_info, latest_snapshot_hashes: None,