GeorryHuang commented on code in PR #2924:
URL: https://github.com/apache/hbase/pull/2924#discussion_r1378327906
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/ModifyTableProcedure.java:
##########
@@ -125,7 +185,11 @@ protected Flow executeFromState(final MasterProcedureEnv
env, final ModifyTableS
break;
case MODIFY_TABLE_PRE_OPERATION:
preModify(env, state);
- setNextState(ModifyTableState.MODIFY_TABLE_CLOSE_EXCESS_REPLICAS);
+ if (reopenRegions) {
+ setNextState(ModifyTableState.MODIFY_TABLE_CLOSE_EXCESS_REPLICAS);
Review Comment:
There are two states in the main procedure process that skip handling
replica-related management when 'reopenRegions' is set to 'false'. I've added a
comment in the first state.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
