morningman commented on a change in pull request #2781: Batch Operate Rollup
table in Doris #2671
URL: https://github.com/apache/incubator-doris/pull/2781#discussion_r376701214
##########
File path: fe/src/main/java/org/apache/doris/persist/OperationType.java
##########
@@ -56,6 +56,7 @@
public static final short OP_FINISH_CONSISTENCY_CHECK = 29;
public static final short OP_RENAME_ROLLUP = 120;
public static final short OP_ALTER_JOB_V2 = 121;
+ public static final short OP_BATCH_ALTER_JOB_V2 = 123;
Review comment:
put the after the `OP_MODIFY_DISTRIBUTION_TYPE` to make the opCode ordered,
even if `OP_BATCH_ALTER_JOB_V2` is the same catalog of ALTER JOB. Otherwise,
the following developer may use wrong code.
----------------------------------------------------------------
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
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
