morningman commented on code in PR #16335:
URL: https://github.com/apache/doris/pull/16335#discussion_r1103531027
##########
gensrc/thrift/FrontendService.thrift:
##########
@@ -716,6 +717,23 @@ struct TFetchSchemaTableDataResult {
2: optional list<Data.TRow> data_batch;
}
+// Only support base table add columns
+struct TAddColumnsRequest {
+ 1: required i64 table_id
+ 2: required list<TColumnDef> addColumns
+ 3: optional string table_name
+ 4: optional string db_name
+ 5: optional bool allow_type_conflict
+}
+
+// Only support base table add columns
+struct TAddColumnsResult {
+ 1: required Status.TStatus status
+ 2: required i64 table_id
+ 3: required list<Descriptors.TColumn> allColumns
+ 4: required i32 schema_version
Review Comment:
Use optional
--
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
