fix naming and doc for maybe_snapshot_abort (#478)

Signed-off-by: Arnaud Gourlay <arnaud.gourlay@gmail.com>

Author: agourlay

src/raft.rs

@@ -1766,10 +1766,10 @@ impl<T: Storage> Raft<T> {
         match pr.state {
             ProgressState::Probe => pr.become_replicate(),
             ProgressState::Snapshot => {
-                if pr.maybe_snapshot_abort() {
+                if pr.is_snapshot_caught_up() {
                     debug!(
                         self.r.logger,
-                        "snapshot aborted, resumed sending replication messages to {from}",
+                        "snapshot caught up, resumed sending replication messages to {from}",
                         from = m.from;
                         "progress" => ?pr,
                     );

src/tracker/progress.rs

@@ -125,10 +125,9 @@ impl Progress {
         self.pending_snapshot = 0;
     }
 
-    /// Unsets pendingSnapshot if Match is equal or higher than
-    /// the pendingSnapshot
+    /// Returns true if Match is equal or higher than the pendingSnapshot.
     #[inline]
-    pub fn maybe_snapshot_abort(&self) -> bool {
+    pub fn is_snapshot_caught_up(&self) -> bool {
         self.state == ProgressState::Snapshot && self.matched >= self.pending_snapshot
     }