fjtirado commented on code in PR #3419:
URL:
https://github.com/apache/incubator-kie-kogito-runtimes/pull/3419#discussion_r1506484147
##########
jbpm/jbpm-flow/src/main/java/org/jbpm/workflow/instance/NodeInstanceContainer.java:
##########
@@ -63,7 +63,7 @@ default NodeInstance getByNodeDefinitionId(final String
nodeDefinitionId, NodeCo
for (Node node : nodeContainer.getNodes()) {
if (nodeDefinitionId.equals(node.getMetaData().get(UNIQUE_ID))) {
- return getNodeInstance(node);
+ return ((NodeInstanceContainer) getNodeInstance((Node)
nodeContainer)).getNodeInstance(node);
Review Comment:
This fixed the NodeNotFoundException. which happens when the failed node is
contained within a compositecontexNode. The rest of the PR fixes the patch
issue (needed to actually fix the model before retry), which was replacing
rather than merging the whole model
--
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]
