Repository: hbase
Updated Branches:
  refs/heads/master 54e8c4bf2 -> 8765ffb0c


HBASE-14391 Empty regionserver WAL will never be deleted although the 
coresponding regionserver has been stale (Qianxi Zhang)


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/8765ffb0
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/8765ffb0
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/8765ffb0

Branch: refs/heads/master
Commit: 8765ffb0ca2b1c39d864ac495f0b3ee2abc520ed
Parents: 54e8c4b
Author: Jerry He <jerry...@apache.org>
Authored: Mon Sep 21 15:03:43 2015 -0700
Committer: Jerry He <jerry...@apache.org>
Committed: Mon Sep 21 15:03:43 2015 -0700

----------------------------------------------------------------------
 .../java/org/apache/hadoop/hbase/master/MasterFileSystem.java    | 4 ++++
 1 file changed, 4 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/8765ffb0/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterFileSystem.java
----------------------------------------------------------------------
diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterFileSystem.java
 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterFileSystem.java
index 9b95ae7..2037786 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterFileSystem.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterFileSystem.java
@@ -251,6 +251,10 @@ public class MasterFileSystem {
           FileStatus[] curLogFiles = FSUtils.listStatus(this.fs, 
status.getPath(), null);
           if (curLogFiles == null || curLogFiles.length == 0) {
             // Empty log folder. No recovery needed
+            // Just delete the directory
+            if (!fs.delete(status.getPath(), false)) {
+              LOG.warn("Unable to delete " + status.getPath());
+            }
             continue;
           }
           final ServerName serverName = 
DefaultWALProvider.getServerNameFromWALDirectoryName(

Reply via email to