SWJTU-ZhangLei commented on code in PR #17207:
URL: https://github.com/apache/doris/pull/17207#discussion_r1123108226
##########
fe/fe-core/src/main/java/org/apache/doris/persist/EditLog.java:
##########
@@ -869,7 +869,7 @@ public static void loadJournal(Env env, JournalEntity
journal) {
env.getCatalogMgr().replayRefreshCatalog(log);
break;
}
- case OperationType.OP_MODIFY_TABLE_ADD_OR_DROP_COLUMNS: {
+ case
OperationType.OP_MODIFY_TABLE_ADD_OR_DROP_OR_MODIFY_COLUMNS: {
Review Comment:
> I just changed the name, what do you think about the name
OP_MODIFY_TABLE_LIGHT_SCHEMA_CHANGE?
1、you are right! Just changed the name will not cause compatibility problem.
2、 OP_MODIFY_TABLE_LIGHT_SCHEMA_CHANGE is a more comfortable name, the
function name
how about replace 'modifyTableAddOrDropOrModifyColumns()' with
'modifyTableLighSchemaChange()'?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
