[GitHub] [incubator-doris] imay commented on a change in pull request #1476: We can set param 'load_memtable_bytes' with http header for stream load

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

imay commented on a change in pull request #1476: We can set param 'load_memtable_bytes' with http header for stream load
URL: https://github.com/apache/incubator-doris/pull/1476#discussion_r303312293
 
 

 ##########
 File path: gensrc/thrift/PaloInternalService.thrift
 ##########
 @@ -176,6 +176,8 @@ struct TPlanFragmentExecParams {
   9: optional i32 sender_id
   10: optional i32 num_senders
   11: optional bool send_query_statistics_with_every_batch
+  // Set MemTable used memory size in stream load
+  12: optional i64 load_memtable_bytes;
 
 Review comment:
   I think it is better to move this param to OlapTableSink?

----------------------------------------------------------------
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


With regards,
Apache Git Services

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