From 22fc3374495770b4fc64e2618344b0798b45e2b0 Mon Sep 17 00:00:00 2001 From: rob-maron <132852777+rob-maron@users.noreply.github.com> Date: Thu, 14 Dec 2023 18:58:52 -0500 Subject: [PATCH] [Stability] Disable saving of VID shares (for now) (#2238) * disable saving of VID shares (for now) * add to RUSTSEC ignore --- task-impls/src/consensus.rs | 22 +++++++++++++--------- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/task-impls/src/consensus.rs b/task-impls/src/consensus.rs index 7b06804e53..3f43311c45 100644 --- a/task-impls/src/consensus.rs +++ b/task-impls/src/consensus.rs @@ -231,15 +231,17 @@ impl, A: ConsensusApi + } } + // TODO: re-enable this when HotShot/the sequencer needs the shares for something + // issue: https://github.com/EspressoSystems/HotShot/issues/2236 // Only vote if you has seen the VID share for this view - if let Some(_vid_share) = self.vid_shares.get(&proposal.view_number) { - } else { - debug!( - "We have not seen the VID share for this view {:?} yet, so we cannot vote.", - proposal.view_number - ); - return false; - } + // if let Some(_vid_share) = self.vid_shares.get(&proposal.view_number) { + // } else { + // debug!( + // "We have not seen the VID share for this view {:?} yet, so we cannot vote.", + // proposal.view_number + // ); + // return false; + // } // Only vote if you have the DA cert // ED Need to update the view number this is stored under? @@ -952,7 +954,9 @@ impl, A: ConsensusApi + .await; // Add to the storage that we have received the VID disperse for a specific view - self.vid_shares.insert(view, disperse); + // TODO: re-enable this when HotShot/the sequencer needs the shares for something + // issue: https://github.com/EspressoSystems/HotShot/issues/2236 + // self.vid_shares.insert(view, disperse); } HotShotEvent::ViewChange(new_view) => { debug!("View Change event for view {} in consensus task", *new_view);