[GitHub] [incubator-doris] chaoyli commented on pull request #4143: Fix alter schema add key column bug in agg model

[GitBox <gi...@apache.org>]

chaoyli commented on pull request #4143:
URL: https://github.com/apache/incubator-doris/pull/4143#issuecomment-663314921


   Now, Add key column in prefix index will do directly schema change.
   ```
   if (base_tablet->num_short_key_columns() != new_tablet->num_short_key_columns()) {
     // the number of short_keys changed, can't do linked schema change
     *sc_directly = true;
     return OLAP_SUCCESS;
   }
   ```
   Another add key column will not change the order of record, it will be ok to do linked schema change.
   So you should check why linked schema change will failed on zone_map.
   You can reference the following function which is used to handle the zone map upon linked schema change.
   ```
   add_rowset_for_linked_schema_change
   ```


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org