GeorryHuang commented on code in PR #2924:
URL: https://github.com/apache/hbase/pull/2924#discussion_r1378329971
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/ModifyTableProcedure.java:
##########
@@ -77,14 +80,22 @@ public ModifyTableProcedure(final MasterProcedureEnv env,
final TableDescriptor
public ModifyTableProcedure(final MasterProcedureEnv env, final
TableDescriptor htd,
final ProcedurePrepareLatch latch) throws HBaseIOException {
- this(env, htd, latch, null, false);
+ this(env, htd, latch, null, false, true);
}
public ModifyTableProcedure(final MasterProcedureEnv env,
final TableDescriptor newTableDescriptor, final ProcedurePrepareLatch
latch,
final TableDescriptor oldTableDescriptor, final boolean
shouldCheckDescriptor)
throws HBaseIOException {
+ this(env, newTableDescriptor, latch, oldTableDescriptor,
shouldCheckDescriptor, true);
+ }
+
+ public ModifyTableProcedure(final MasterProcedureEnv env,
Review Comment:
Removed the new constructor I added before and altered the existing
constructor instead.
--
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]
