[Fix-17597][Workflow] fix condition task node not run when predecessor node failed

[Mrhs121]

…r node failed

Purpose of the pull request

close #17597

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

Pull Request Notice

Pull Request Notice

If your pull request contains incompatible change, you should also add it to docs/docs/en/guide/upgrade/incompatible.md

[Mrhs121]

@ruanwenjun cc, please help review it when you have time, thank you.

[ruanwenjun]

This is caused by the judge logic of IWorkflowExecutionGraph#isEndOfTaskChain is incorrect.

[Mrhs121]

This is caused by the judge logic of IWorkflowExecutionGraph#isEndOfTaskChain is incorrect.

The judgment logic in the isTriggerConditionMet method also has the same problem. If the predecessor node that the condition node depends on fails, it will return early.

dolphinscheduler/dolphinscheduler-master/src/main/java/org/apache/dolphinscheduler/server/master/engine/workflow/statemachine/AbstractWorkflowStateAction.java

Lines 72 to 82 in f4646ce

	protected void triggerTasks(final IWorkflowExecutionRunnable workflowExecutionRunnable,
	final List<ITaskExecutionRunnable> taskExecutionRunnables) {
	final IWorkflowExecutionGraph workflowExecutionGraph = workflowExecutionRunnable.getWorkflowExecutionGraph();
	final List<ITaskExecutionRunnable> readyTaskExecutionRunnableList = taskExecutionRunnables
	.stream()
	.filter(workflowExecutionGraph::isTriggerConditionMet)
	.collect(Collectors.toList());
	if (CollectionUtils.isEmpty(readyTaskExecutionRunnableList)) {
	emitWorkflowFinishedEventIfApplicable(workflowExecutionRunnable);
	return;
	}

[ruanwenjun]

duplicated with #17606