-
Notifications
You must be signed in to change notification settings - Fork 555
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
Migrate child process instances #15922
Comments
Checked if there are any properties of the call activity instance that should be updated during the migration of the called child process instance. It turned out that it is not necessary. Only following fields of the child process instance can change:
Checked following classes to see if anything needs to be updated for the calling call activity: |
## Description We've verified that the call activity that is referencing the migrated child process instance does not have any properties that needs to be updated on migration. Following fields can be changed during the migration of child process instance: - the process definition key of the child process instance - the bpmn process id of the child process instance - the version of the child process instance - the element id of the active element inside the child process instance Checked following classes to see if anything needs to be updated for the calling call activity: `ProcessInstanceRecord` -> no data kept for child process instance `ElementInstance` -> only called child instance's key is kept but it will not change during the migration `DBElementInstanceState` -> parent and child instances are connected through instance keys that will not change during the migration of child process instance Therefore, there is no need to update call activity neither in log stream nor in the state. Related issues #18328 closes #15922
# Description Backport of #18509 to `stable/8.5`. relates to #18328 #15922 original author: @berkaycanbc
Similar to #15921, we also want to support migrating active child process instances, i.e. process instances called from a call activity.
This would be useful because it expands the coverage of process instances that can be migrated.
When working on this issue, please consider whether the call activity instance must be updated in any way. If so, we may need to add a new
ProcessInstance:CHILD_MIGRATED
event. Re-using theProcessInstance:ELEMENT_MIGRATED
for the call activity instance would be wrong semantically, as the call activity instance wasn't migrated.The text was updated successfully, but these errors were encountered: