Lchangliang commented on code in PR #12160:
URL: https://github.com/apache/doris/pull/12160#discussion_r965458379
##########
be/src/olap/tablet_schema.cpp:
##########
@@ -465,6 +466,11 @@ void TabletSchema::append_column(TabletColumn column, bool
is_dropped_column) {
if (column.is_nullable()) {
_num_null_columns++;
}
+ if (UNLIKELY(column.name() == DELETE_SIGN)) {
+ _delete_sign_idx = _num_columns;
+ } else if (UNLIKELY(column.name() == SEQUENCE_COL)) {
+ _sequence_col_idx = _num_columns;
+ }
Review Comment:
I can't find the alter statement in doris doc. But it doesn't match after
light schema change, because the hidden columns are always in the end. So when
read and load, BE need to set their idx.
--
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: commits-unsubscr...@doris.apache.org
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
