Remove unused TaskError structs (#2637)

crates/task-impls/src/consensus.rs

@@ -39,7 +39,6 @@ use tracing::warn;
 
 use crate::vote::HandleVoteEvent;
 use chrono::Utc;
-use snafu::Snafu;
 use std::{
     collections::{BTreeMap, HashSet},
     marker::PhantomData,
@@ -49,10 +48,6 @@ use std::{
 use tokio::task::JoinHandle;
 use tracing::{debug, error, info, instrument};
 
-/// Error returned by the consensus task
-#[derive(Snafu, Debug)]
-pub struct ConsensusTaskError {}
-
 /// Alias for the block payload commitment and the associated metadata.
 pub struct CommitmentAndMetadata<PAYLOAD: BlockPayload> {
     /// Vid Commitment

crates/task-impls/src/da.rs

@@ -28,17 +28,12 @@ use hotshot_types::{
 use sha2::{Digest, Sha256};
 
 use crate::vote::HandleVoteEvent;
-use snafu::Snafu;
 use std::{marker::PhantomData, sync::Arc};
 use tracing::{debug, error, instrument, warn};
 
 /// Alias for Optional type for Vote Collectors
 type VoteCollectorOption<TYPES, VOTE, CERT> = Option<VoteCollectionTaskState<TYPES, VOTE, CERT>>;
 
-#[derive(Snafu, Debug)]
-/// Error type for consensus tasks
-pub struct ConsensusTaskError {}
-
 /// Tracks state of a DA task
 pub struct DATaskState<
     TYPES: NodeType,

crates/task-impls/src/transactions.rs

@@ -25,7 +25,6 @@ use hotshot_types::{
     },
 };
 use hotshot_utils::bincode::bincode_opts;
-use snafu::Snafu;
 use std::{
     collections::{HashMap, HashSet},
     sync::Arc,
@@ -36,10 +35,6 @@ use tracing::{debug, error, instrument, warn};
 /// A type alias for `HashMap<Commitment<T>, T>`
 type CommitmentMap<T> = HashMap<Commitment<T>, T>;
 
-#[derive(Snafu, Debug)]
-/// Error type for consensus tasks
-pub struct ConsensusTaskError {}
-
 /// Tracks state of a Transaction task
 pub struct TransactionTaskState<
     TYPES: NodeType,

crates/task-impls/src/upgrade.rs

@@ -21,17 +21,12 @@ use hotshot_types::{
 };
 
 use crate::vote::HandleVoteEvent;
-use snafu::Snafu;
 use std::sync::Arc;
 use tracing::{debug, error, instrument, warn};
 
 /// Alias for Optional type for Vote Collectors
 type VoteCollectorOption<TYPES, VOTE, CERT> = Option<VoteCollectionTaskState<TYPES, VOTE, CERT>>;
 
-#[derive(Snafu, Debug)]
-/// Error type for consensus tasks
-pub struct ConsensusTaskError {}
-
 /// Tracks state of a DA task
 pub struct UpgradeTaskState<
     TYPES: NodeType,

crates/task-impls/src/vid.rs

@@ -25,15 +25,10 @@ use hotshot_types::{
 #[cfg(async_executor_impl = "tokio")]
 use tokio::task::spawn_blocking;
 
-use snafu::Snafu;
 use std::marker::PhantomData;
 use std::sync::Arc;
 use tracing::{debug, error, instrument, warn};
 
-#[derive(Snafu, Debug)]
-/// Error type for consensus tasks
-pub struct ConsensusTaskError {}
-
 /// Tracks state of a VID task
 pub struct VIDTaskState<
     TYPES: NodeType,

crates/task-impls/src/view_sync.rs

@@ -35,7 +35,6 @@ use hotshot_types::{
         node_implementation::{ConsensusTime, NodeImplementation, NodeType},
     },
 };
-use snafu::Snafu;
 use std::{collections::BTreeMap, collections::HashMap, fmt::Debug, sync::Arc, time::Duration};
 #[cfg(async_executor_impl = "tokio")]
 use tokio::task::JoinHandle;
@@ -53,10 +52,6 @@ pub enum ViewSyncPhase {
     Finalize,
 }
 
-#[derive(Snafu, Debug)]
-/// Stub of a view sync error
-pub struct ViewSyncTaskError {}
-
 /// Type alias for a map from View Number to Relay to Vote Task
 type RelayMap<TYPES, VOTE, CERT> =
     HashMap<<TYPES as NodeType>::Time, BTreeMap<u64, VoteCollectionTaskState<TYPES, VOTE, CERT>>>;

crates/task-impls/src/vote.rs

@@ -21,13 +21,8 @@ use hotshot_types::{
     traits::{election::Membership, node_implementation::NodeType},
     vote::{Certificate, HasViewNumber, Vote, VoteAccumulator},
 };
-use snafu::Snafu;
 use tracing::{debug, error};
 
-#[derive(Snafu, Debug)]
-/// Stub of a vote error
-pub struct VoteTaskError {}
-
 /// Task state for collecting votes of one type and emiting a certificate
 pub struct VoteCollectionTaskState<
     TYPES: NodeType,