[GitHub] [doris] morningman commented on pull request #18108: [Enhancement](Broker load) Show accurate import progress of broker load

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

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

   Still not clear.
   We should unify all kinds of load.
   
   1. The total source file size is known, for all kinds of load(stream, broker, etc.)
   2. The current read bytes should be got from `IOContext`, instead of adding a new parameter.
   3. The progress is "read bytes/total bytes * 100%"
   
   And the modified code should only related to `file reader` and `IOContext`
   
   


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