[Improvement-18127][Master] Remove the redundant workflow completion check when handling workflow timeout events (#18128)

Author: njnu-seafish

dolphinscheduler-master/src/main/java/org/apache/dolphinscheduler/server/master/engine/workflow/lifecycle/handler/WorkflowTimeoutLifecycleEventHandler.java

@@ -19,7 +19,6 @@
 
 import org.apache.dolphinscheduler.dao.entity.WorkflowInstance;
 import org.apache.dolphinscheduler.server.master.engine.ILifecycleEventType;
-import org.apache.dolphinscheduler.server.master.engine.graph.IWorkflowExecutionGraph;
 import org.apache.dolphinscheduler.server.master.engine.workflow.lifecycle.WorkflowLifecycleEventType;
 import org.apache.dolphinscheduler.server.master.engine.workflow.lifecycle.event.WorkflowTimeoutLifecycleEvent;
 import org.apache.dolphinscheduler.server.master.engine.workflow.runnable.IWorkflowExecutionRunnable;
@@ -46,12 +45,6 @@ public WorkflowTimeoutLifecycleEventHandler(final WorkflowAlertManager workflowA
     public void handle(final IWorkflowStateAction workflowStateAction,
                        final IWorkflowExecutionRunnable workflowExecutionRunnable,
                        final WorkflowTimeoutLifecycleEvent workflowTimeoutLifecycleEvent) {
-        final IWorkflowExecutionGraph workflowExecutionGraph = workflowExecutionRunnable.getWorkflowExecutionGraph();
-        if (workflowExecutionGraph.isAllTaskExecutionRunnableChainFinish()) {
-            // all the TaskExecutionRunnable chain in the graph is finish, means the workflow is already finished.
-            return;
-        }
-
         final WorkflowInstance workflowInstance = workflowExecutionRunnable.getWorkflowInstance();
         final boolean shouldSendAlert = workflowInstance.getWarningGroupId() != null;