commit 2317f13d71c6670aed9ace967decef53d9c994dc
author Tak Lon (Stephen) Wu <taklwu@apache.org> Wed Jun 02 14:19:21 2021 -0700
committer GitHub <noreply@github.com> Wed Jun 02 14:19:21 2021 -0700
tree 05a95c6df2ae8d8a8b7d5edf907886d2781bc2c0
parent 22458706ffc8973d6bd78ed325c138321cd19927

HBASE-25965 Move delete of WAL directory out of LOG.info and avoid leftover directory (#89)

Signed-off-by: Anoop Sam John <anoopsamjohn@apache.org>

hbase-hbck2/src/main/java/org/apache/hbase/hbck1/HBaseFsck.java

diff --git a/hbase-hbck2/src/main/java/org/apache/hbase/hbck1/HBaseFsck.java b/hbase-hbck2/src/main/java/org/apache/hbase/hbck1/HBaseFsck.java
index 256ff51..8e019b8 100644
--- a/hbase-hbck2/src/main/java/org/apache/hbase/hbck1/HBaseFsck.java
+++ b/hbase-hbck2/src/main/java/org/apache/hbase/hbck1/HBaseFsck.java
@@ -1930,8 +1930,8 @@
     } finally {
       HBaseTestingUtility.closeRegionAndWAL(meta);
       // Clean out the WAL we created and used here.
-      LOG.info("Deleting {}, result={}", waldir,
-          HBCKFsUtils.delete(waldir.getFileSystem(getConf()), waldir, true));
+      boolean deleteWalDir = HBCKFsUtils.delete(waldir.getFileSystem(getConf()), waldir, true);
+      LOG.info("Deleting WAL directory {}, result={}", waldir, deleteWalDir);
     }
     LOG.info("Success! hbase:meta table rebuilt. Old hbase:meta moved into " + backupDir);
     return true;