-
Notifications
You must be signed in to change notification settings - Fork 558
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(engine): migrate sub-process processor #4590
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but just a reminder for a todo you left earlier
@@ -56,37 +147,4 @@ public void onChildTerminated( | |||
eventSubscriptionBehavior.publishTriggeredEventSubProcess(flowScopeContext); | |||
} | |||
} | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you don't mean to forget the todo on line 129, so here's a reminder :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I removed the TODO and added a comment on the state issue :)
4591: [Backport] Interrupt Sub-Process with parallel gateway r=saig0 a=saig0 ## Description * publish only deferred boundary events when the sub-process is terminated to avoid that other events are published (e.g. a taken sequence flow on a joining parallel gateway) ## Related issues Backport of #4590 # Co-authored-by: Philipp Ossler <philipp.ossler@gmail.com>
4592: [Backport] Interrupt Sub-Process with parallel gateway r=saig0 a=saig0 ## Description * publish only deferred boundary events when the sub-process is terminated to avoid that other events are published (e.g. a taken sequence flow on a joining parallel gateway) ## Related issues Backport of #4590 # Co-authored-by: Philipp Ossler <philipp.ossler@gmail.com>
* fix termination of an embedded sub-process with a waiting token on a joining parallel gateway * clean up tests for embedded sub-process
13e28de
to
7de4c75
Compare
bors r+ |
4590: chore(engine): migrate sub-process processor r=saig0 a=saig0 # Description * migrate sub-process processor * fix termination of an embedded sub-process with a waiting token on a joining parallel gateway * clean up tests for embedded sub-process ## Related issues closes #4474 closes #4400 closes #4352 # Co-authored-by: Philipp Ossler <philipp.ossler@gmail.com>
Build failed |
bors retry |
4590: chore(engine): migrate sub-process processor r=saig0 a=saig0 # Description * migrate sub-process processor * fix termination of an embedded sub-process with a waiting token on a joining parallel gateway * clean up tests for embedded sub-process ## Related issues closes #4474 closes #4400 closes #4352 # Co-authored-by: Philipp Ossler <philipp.ossler@gmail.com>
Build failed |
bors r+ |
4590: chore(engine): migrate sub-process processor r=saig0 a=saig0 # Description * migrate sub-process processor * fix termination of an embedded sub-process with a waiting token on a joining parallel gateway * clean up tests for embedded sub-process ## Related issues closes #4474 closes #4400 closes #4352 # Co-authored-by: Philipp Ossler <philipp.ossler@gmail.com>
Build failed |
bors r+ ..please 💰 |
Build succeeded |
Description
Related issues
closes #4474
closes #4400
closes #4352
Pull Request Checklist
mvn clean install -DskipTests
locally before committing