slfan1989 commented on code in PR #1227:
URL: https://github.com/apache/ratis/pull/1227#discussion_r2098899033


##########
ratis-server/src/test/java/org/apache/ratis/InstallSnapshotNotificationTests.java:
##########
@@ -220,7 +220,7 @@ private void testAddNewFollowers(CLUSTER cluster, int 
numRequests) throws Except
     // delete the log segments from the leader
     LOG.info("Delete logs {}", logs);
     for (LogSegmentPath path : logs) {
-      FileUtils.deleteFully(path.getPath()); // the log may be already purged
+      FileUtils.deleteFully(path.getPath()); // the log may be already puged

Review Comment:
   Thank you very much for reviewing the code! This issue was indeed an 
oversight on my part. I will fix it along with the flaky tests in the upcoming 
revisions.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to