I can filter process instances by root process instance id

[ThorbenLindhauer]

This issue was imported from JIRA:

Field	Value
JIRA Link	CAM-13424
Reporter	@ThorbenLindhauer
Has restricted visibility comments	true

---

User Story (Required on creation):

In order to suspend/activate an entire process instance hierarchy at once, the IDs of those instances must be known. If I can filter by root process instance id, I can easily do that.

Functional Requirements (Required before implementation):

• There is a new filter on the process instance query rootProcessInstanceId that returns all instances with a root process instance id equal to the argument

Technical Requirements (Required before implementation):

• Java API, REST API, OpenAPI, Cockpit

Limitations of Scope (Optional):

Hints (Optional):

• Consider implementing it for the historic query as well to be consistent

Links:

• is depended on by https://jira.camunda.com/browse/SUPPORT-10034

Pull Requests

Preview Give feedback

1. Implement filteration by rootProcessInstanceId for Process Instances and History Process instances endpoints #2582 #4925
   +0
   

[prakashpalanisamy]

@ThorbenLindhauer I would like to pick this issue up. Please let me know if that okay with you. Thanks!!

[joaquinfelici]

Hello @prakashpalanisamy ,

Thanks for your interest in contributing. Indeed, you can go ahead and work on this, and open a pull request when you think it's ready.

Something very similar was done for the historic task instance query api, you can take inspiration from it.

Some pointers and useful links:

• createProcessInstanceQuery() endpoint.
• ProcessInstanceQuery class (public java api).
• ProcessInstanceQueryImpl (implementation).
• ProcessInstanceQueryDto (rest api).
• Both tables ACT_RU_EXECUTION and ACT_HI_PROCINST (in case you also decide to tackle the historic api which is optional) already have column ROOT_PROC_INST_ID_ (more info about database schema here).
• Don't forget to add tests for the new implementation.
• Don't forget to adapt documentation for both java and rest api (was also done in the example I provided above, you can check how).

Thanks again!

[prakashpalanisamy]

@joaquinfelici Thank you for all the pointers that you have provided and giving me a go ahead to take this issue up. Ill take some time to go over the links and get back if there are any questions. Thank you!! :)

[prakashpalanisamy]

@joaquinfelici I have raised a PR for this Git Issue #4925 . Can you please take a look and provide your feedback when you get time? Thanks!!

[joaquinfelici]

Hey @prakashpalanisamy , thanks! I left a couple of comments in the PR, will continue the review after the CI is finished.