yiguolei commented on code in PR #16940:
URL: https://github.com/apache/doris/pull/16940#discussion_r1113783721
##########
be/src/runtime/fragment_mgr.cpp:
##########
@@ -330,7 +330,9 @@ std::string FragmentMgr::to_http_path(const std::string&
file_name) {
void FragmentMgr::coordinator_callback(const ReportStatusRequest& req) {
DCHECK(req.status.ok() || req.done); // if !status.ok() => done
Status exec_status = req.update_fn(req.status);
-
+ if (_exec_env->new_load_stream_mgr()->have_promise(req.query_id)) {
Review Comment:
do not depend on this to set promise. coordinator_callback only means the
fragment is finished on be, but there are some state on fe.
Add a rpc method in be, and if fe find the coord finished, then fe call this
rpc service to indicate the load finished.
--
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
