zddr commented on code in PR #48830:
URL: https://github.com/apache/doris/pull/48830#discussion_r2018487878


##########
fe/fe-core/src/main/java/org/apache/doris/job/extensions/mtmv/MTMVTask.java:
##########
@@ -364,6 +364,14 @@ public synchronized boolean cancel(boolean 
needWaitCancelComplete) throws JobExc
 
     @Override
     protected void executeCancelLogic(boolean needWaitCancelComplete) {
+        try {
+            if (mtmv == null) {

Review Comment:
   Mtmv is initialized in the before method. If the task has not yet run, the 
before method will not be used, so mtmv will be empty, which prevents the 
canceled task from being added to the history list



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

Reply via email to