Change: remove AddLearnerError::Exists, which is not actually used

datafuselabs · Sep 30, 2022 · 1bd22ed · 1bd22ed
1 parent 2c06dc4
commit 1bd22ed
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 7 deletions.
diff --git a/openraft/src/error.rs b/openraft/src/error.rs
@@ -115,9 +115,6 @@ pub enum AddLearnerError {
     #[error(transparent)]
     ForwardToLeader(#[from] ForwardToLeader),
 
-    #[error("node {0} is already a learner")]
-    Exists(NodeId),
-
     #[error(transparent)]
     Fatal(#[from] Fatal),
 }

diff --git a/openraft/src/raft.rs b/openraft/src/raft.rs
@@ -317,10 +317,6 @@ impl<D: AppData, R: AppDataResponse, N: RaftNetwork<D>, S: RaftStorage<D, R>> Ra
                 AddLearnerError::ForwardToLeader(forward_err) => {
                     return Err(ClientWriteError::ForwardToLeader(forward_err))
                 }
-                AddLearnerError::Exists(node_id) => {
-                    tracing::info!(%node_id, "add learner: already exists");
-                    continue;
-                }
                 AddLearnerError::Fatal(f) => return Err(ClientWriteError::Fatal(f)),
             }
         }