yiguolei commented on code in PR #66582:
URL: https://github.com/apache/doris/pull/66582#discussion_r3735680469
##########
fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java:
##########
@@ -1042,9 +1043,73 @@ public void finalizeQuery() {
// received after unregisterQuery(), causing the instance profile to
be lost, so we should wait
// for the profile before unregisterQuery().
updateProfile(true);
+
+ // After the query finishes on FE, the BE may still hold the
QueryContext alive via
+ // _query_ctx_map_delay_delete (inserted when runtime filter merge
controllers are non-empty).
+ // Profile reporting (_report_query_profile) is bound to QueryContext
destructor, so the
+ // final BE profile RPC may not have arrived yet. We wait here so that
the Coordinator stays
+ // registered long enough for the incoming profile RPC to be processed
via reportExecStatus().
+ waitForProfileComplete();
Review Comment:
你不能这么写，这么写，如果我们开了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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
