taklwu commented on a change in pull request #2237:
URL: https://github.com/apache/hbase/pull/2237#discussion_r469554017



##########
File path: 
hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
##########
@@ -915,6 +917,11 @@ private void 
finishActiveMasterInitialization(MonitoredTask status)
       this.tableDescriptors.getAll();
     }
 
+    // check cluster Id stored in ZNode before, and use it to indicate if a 
cluster has been
+    // restarted with an existing Zookeeper quorum.
+    isClusterRestartWithExistingZNodes =

Review comment:
       > So this says the need to keep this boolean info somewhere once we find 
that and even before creating the zk node for ClusterId. Am I making the 
concern clear this time?
   
   ack and I got the concern. 
   
   but before I move to implement the proc level information, let's wait a bit 
on @Apache9 for the question on META's `tableInfo` and `partial meta`.  The 
ideal case is that we may be able to use 
`FSTableDescriptors.getTableDescriptorFromFs()` if found and can be readed to 
indicate meta is not partial. 




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