deardeng commented on code in PR #57013:
URL: https://github.com/apache/doris/pull/57013#discussion_r2513984871


##########
fe/fe-core/src/main/java/org/apache/doris/cloud/alter/CloudSchemaChangeHandler.java:
##########
@@ -91,6 +93,7 @@ public void updatePartitionsProperties(Database db, String 
tableName, List<Strin
         }
     }
 
+    //TODO: extract the common code with SchemaChangeHandler
     @Override

Review Comment:
   咦?这里为啥不在SchemaChangeHandler 实现?这个功能是cloud专有?



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

Reply via email to