dataroaring commented on code in PR #39756:
URL: https://github.com/apache/doris/pull/39756#discussion_r1732844104
##########
gensrc/thrift/Descriptors.thrift:
##########
@@ -248,12 +248,14 @@ struct TOlapTableSchemaParam {
5: required TTupleDescriptor tuple_desc
6: required list<TOlapTableIndexSchema> indexes
7: optional bool is_dynamic_schema // deprecated
- 8: optional bool is_partial_update
+ 8: optional bool is_partial_update // deprecated, use
unique_key_update_mode
9: optional list<string> partial_update_input_columns
10: optional bool is_strict_mode = false
11: optional string auto_increment_column
12: optional i32 auto_increment_column_unique_id = -1
13: optional Types.TInvertedIndexFileStorageFormat
inverted_index_file_storage_format = Types.TInvertedIndexFileStorageFormat.V1
+ 14: optional Types.TUniqueKeyUpdateMode unique_key_update_mode =
Types.TUniqueKeyUpdateMode.UPSERT
+ 15: optional i32 sequence_map_col_unique_id = -1
Review Comment:
Maybe merge_bitmap is a better name.
--
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]
