[GitHub] [doris] morningman commented on pull request #18291: [fix](profile) fix problem on stream load profile log

["morningman (via GitHub)" <gi...@apache.org>]

morningman commented on PR #18291:
URL: https://github.com/apache/doris/pull/18291#issuecomment-1493717919

   I think profile for stream load is necessary, but not a good idea to use a globle BE config `enable_stream_load_profile_log` to control it.
   BYW, there is no `session context` for a stream load, so we can't get session variable when doing stream load.
   A proper way should is to add a new param for stream load, eg "`-H enable_profile:true`" to control each stream load job
   whether to print profile.


-- 
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-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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