anoopsjohn commented on a change in pull request #2113:
URL: https://github.com/apache/hbase/pull/2113#discussion_r460382240



##########
File path: 
hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/InitMetaProcedure.java
##########
@@ -71,8 +71,8 @@ private static void writeFsLayout(Path rootDir, Configuration 
conf) throws IOExc
     LOG.info("BOOTSTRAP: creating hbase:meta region");
     FileSystem fs = rootDir.getFileSystem(conf);
     Path tableDir = CommonFSUtils.getTableDir(rootDir, 
TableName.META_TABLE_NAME);
-    if (fs.exists(tableDir) && !fs.delete(tableDir, true)) {
-      LOG.warn("Can not delete partial created meta table, continue...");
+    if (fs.exists(tableDir)) {

Review comment:
       This is another issue other than the unknown RS issue.
   In cluster recreate cloud case, the zk data is not there and so no META 
region location. So for HM, this is a cluster bootstrap and so as part of that 
init meta and its FS.  
   HBASE-24471 added this extra code of deleting an existing META table FS 
path.  It is done by considering as a partial created meta table as part of 
some previous failed bootstrap cc @Apache9 
   So this issue is because of the zk data not in new cluster.  Unknown server 
issue is because of loss of WAL (to be precise the master proc wal)




----------------------------------------------------------------
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.

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


Reply via email to