chaoyli commented on a change in pull request #3661: URL: https://github.com/apache/incubator-doris/pull/3661#discussion_r429507733
########## File path: be/src/olap/schema_change.cpp ########## @@ -1859,6 +1859,29 @@ OLAPStatus SchemaChangeHandler::_parse_request(TabletSharedPtr base_tablet, // 若Key列的引用序列出现乱序,则需要重排序 int num_default_value = 0; + // A, B, C are keys(sort keys), D is value + // The following cases are not changing the order, no need to resort: + // (sort keys keep in same order) + // old keys: A B C D + // new keys: A X B C D + // + // old keys: A B C D + // new keys: X A B C D + // + // old keys: A B C D + // new keys: A B C + // + // old keys: A B C D + // new keys: A B Review comment: After this pull request. old keys: A B C D new keys: A B C will be resorted. ---------------------------------------------------------------- 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 --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org