taklwu commented on a change in pull request #2113: URL: https://github.com/apache/hbase/pull/2113#discussion_r461244222
########## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/InitMetaProcedure.java ########## @@ -71,7 +71,11 @@ 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)) { + boolean removeMeta = conf.getBoolean(HConstants.REMOVE_META_ON_RESTART, Review comment: > In cloud redeploy case we will see a pattern where we will have a clusterId in the FS and not in zk. This can be used as an indicator? I will come back on this later tomorrow, but I agreed with you that we should check explicitly how we define *partial bootstrap* and that `partial meta` need some cleanup. also, do you mean if the clusterID did't write to ZK, is it `partial` during bootstrap ? ---------------------------------------------------------------- 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: us...@infra.apache.org