Skip to content

Keep the parent deployment id on the cached process and case definitions - #4307

Merged
filiphr merged 1 commit into
flowable:mainfrom
filiphr:definition-parent-deployment-id
Oct 13, 2026
Merged

filiphr merged 1 commit into
flowable:mainfrom
filiphr:definition-parent-deployment-id

Conversation

@filiphr

@filiphr filiphr commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

The deployers copy the parent deployment id of the deployment onto the cached process and case definitions, so resolving the definition deployment id no longer needs to select the deployment.

The start form, form definitions and decisions lookups of BPMN and CMMN (and the REST start form resources) now use the parent deployment id of the cached definitions instead of their own deployment lookups.

The selects are verified with database operation count tests for a case with a case task, a case with a process task, a process with a case task, a process and a case with a decision task, and getting the decisions of a process and a case definition.

Check List:

  • Unit tests: YES
  • Documentation: NA

The deployers copy the parent deployment id of the deployment onto the cached
process and case definitions, so resolving the definition deployment id no
longer needs to select the deployment.

The start form, form definitions and decisions lookups of BPMN and CMMN (and
the REST start form resources) now use the parent deployment id of the cached
definitions instead of their own deployment lookups.

The selects are verified with database operation count tests for a case with
a case task, a case with a process task, a process with a case task, a process
and a case with a decision task, and getting the decisions of a process and a
case definition.
@filiphr
filiphr merged commit 3cb04ed into flowable:main Oct 13, 2026
3 checks passed
@filiphr
filiphr deleted the definition-parent-deployment-id branch October 13, 2026 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant