morningman commented on a change in pull request #3230:
URL: https://github.com/apache/incubator-doris/pull/3230#discussion_r413914241
##########
File path: gensrc/thrift/BackendService.thrift
##########
@@ -54,11 +55,12 @@ struct TRoutineLoadTask {
6: optional string db
7: optional string tbl
8: optional string label
- 9: optional i64 max_interval_s
- 10: optional i64 max_batch_rows
- 11: optional i64 max_batch_size
- 12: optional TKafkaLoadInfo kafka_load_info
- 13: optional PaloInternalService.TExecPlanFragmentParams params
+ 9: optional PlanNodes.TFileFormatType format
Review comment:
Put new field at the end of the struct
----------------------------------------------------------------
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
