924060929 commented on code in PR #68283:
URL: https://github.com/apache/doris/pull/68283#discussion_r4058646138
##########
fe/fe-core/src/main/java/org/apache/doris/qe/Coordinator.java:
##########
@@ -767,6 +767,9 @@ private boolean shouldQueue() {
// A call to Exec() must precede all other member function calls.
@Override
public void exec() throws Exception {
+ if (isQueryCancelled()) {
Review Comment:
Fixed in 7f78c7361ea. The execution and dispatch fences now reject every
non-OK status, including TIMEOUT, and the statement, legacy coordinator,
Nereids coordinator, and pipeline task tests cover this path. I also fixed the
existing PipelineExecutionTaskTest mocks that failed on the new status read.
--
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]
