Apache9 commented on a change in pull request #1806:
URL: https://github.com/apache/hbase/pull/1806#discussion_r432785688



##########
File path: 
hbase-protocol-shaded/src/main/protobuf/server/master/MasterProcedure.proto
##########
@@ -489,8 +489,9 @@ message ReopenTableRegionsStateData {
 }
 
 enum InitMetaState {
-  INIT_META_ASSIGN_META = 1;
-  INIT_META_CREATE_NAMESPACES = 2;
+  INIT_META_WRITE_FS_LAYOUT = 1;

Review comment:
       I tried to do this in the patch, i.e, adding 'INIT_META_WRITE_FS_LAYOUT 
= 3', but finally I found that it is not necessary, as changing the order or 
adding a new enum will both break the compatibility, so let's just kep the code 
clean, i.e, the order in the enum is the order when executing.




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