Umeshkumar9414 commented on code in PR #7075:
URL: https://github.com/apache/hbase/pull/7075#discussion_r2238693914


##########
hbase-server/src/main/java/org/apache/hadoop/hbase/wal/AbstractRecoveredEditsOutputSink.java:
##########
@@ -130,6 +154,21 @@ protected Path 
closeRecoveredEditsWriterAndFinalizeEdits(RecoveredEditsWriter ed
     return dst;
   }
 
+  private boolean deleteTmpIfDstHasNoLessEntries(RecoveredEditsWriter 
editsWriter, Path dst)
+    throws IOException {
+    if (walSplitter.walFS.exists(dst) && !isDstHasFewerEntries(editsWriter, 
dst)) {
+      LOG.info(
+        "Destination {} already have no fewer entries so deleting tmp 
recovered edits file {}", dst,

Review Comment:
   in latest implementation, removed this new method. 



-- 
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: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to