Skip to content

Commit

Permalink
Simplify executor-consensus interface (MystenLabs#691)
Browse files Browse the repository at this point in the history
Remove useless sync
  • Loading branch information
asonnino authored and huitseeker committed Aug 8, 2022
1 parent 4d78b8d commit 91a76e7
Show file tree
Hide file tree
Showing 9 changed files with 49 additions and 696 deletions.
11 changes: 1 addition & 10 deletions consensus/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -11,14 +11,12 @@ pub mod bullshark;
pub mod consensus;
pub mod dag;
pub mod metrics;
pub mod subscriber;
pub mod tusk;
mod utils;

pub use crate::{consensus::Consensus, subscriber::SubscriberHandler};
pub use crate::consensus::Consensus;

use serde::{Deserialize, Serialize};
use std::ops::RangeInclusive;
use types::{Certificate, SequenceNumber};

/// The default channel size used in the consensus and subscriber logic.
Expand All @@ -32,10 +30,3 @@ pub struct ConsensusOutput {
/// The (global) index associated with this certificate.
pub consensus_index: SequenceNumber,
}

/// The message sent by the client to sync missing chunks of the output sequence.
#[derive(Serialize, Deserialize, Debug)]
pub struct ConsensusSyncRequest {
/// The sequence numbers of the missing consensus outputs.
pub missing: RangeInclusive<SequenceNumber>,
}
128 changes: 0 additions & 128 deletions consensus/src/subscriber.rs

This file was deleted.

Loading

0 comments on commit 91a76e7

Please sign in to comment.