Skip to content

Commit a724265

Browse files
committed
fix: bug#AI Commit#
1 parent 9aebe70 commit a724265

1 file changed

Lines changed: 7 additions & 3 deletions

File tree

  • dss-orchestrator/orchestrators/dss-workflow/dss-flow-execution-server/src/main/scala/com/webank/wedatasphere/dss/flow/execution/entrance/resolver

dss-orchestrator/orchestrators/dss-workflow/dss-flow-execution-server/src/main/scala/com/webank/wedatasphere/dss/flow/execution/entrance/resolver/FlowDependencyResolverImpl.scala

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,9 @@ class FlowDependencyResolverImpl extends FlowDependencyResolver with Logging {
6464
workflowNodesById.get(edge.getSource).exists { sourceNode =>
6565
BranchExpressionUtils.isBranchNode(sourceNode) &&
6666
flowContext.isNodeCompleted(sourceNode.getName) &&
67-
flowJob.hasBranchSelection(sourceNode.getId) &&
68-
!flowJob.isBranchTargetSelected(sourceNode.getId, node.getId)
67+
(flowContext.isNodeSkipped(sourceNode.getName) ||
68+
!flowJob.hasBranchSelection(sourceNode.getId) ||
69+
!flowJob.isBranchTargetSelected(sourceNode.getId, node.getId))
6970
}
7071
}
7172
}
@@ -79,12 +80,15 @@ class FlowDependencyResolverImpl extends FlowDependencyResolver with Logging {
7980
incomingEdges(node).forall { edge =>
8081
workflowNodesById.get(edge.getSource) match {
8182
case Some(sourceNode) if BranchExpressionUtils.isBranchNode(sourceNode) =>
82-
flowJob.hasBranchSelection(sourceNode.getId) && flowJob.isBranchTargetSelected(sourceNode.getId, node.getId)
83+
flowContext.isNodeSucceed(sourceNode.getName) &&
84+
flowJob.hasBranchSelection(sourceNode.getId) &&
85+
flowJob.isBranchTargetSelected(sourceNode.getId, node.getId)
8386
case _ => true
8487
}
8588
}
8689
}
8790

91+
8892
nodes.foreach { node =>
8993
val nodeName = node.getName
9094
def isCanExecutable: Boolean = {

0 commit comments

Comments
 (0)