Skip to content
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

Query all type of Jobs by processDefinitionKey #3692

Closed
remibantos opened this issue Aug 1, 2023 · 0 comments · Fixed by #3783
Closed

Query all type of Jobs by processDefinitionKey #3692

remibantos opened this issue Aug 1, 2023 · 0 comments · Fixed by #3783

Comments

@remibantos
Copy link
Contributor

remibantos commented Aug 1, 2023

Is your feature request related to a problem? Please describe.

I need to get all the deadletter jobs for a specific process definition key, in order to add this number to the total number of errors related to my business process.

As of today, the base job query does not allow to query any type of jobs (job, timer, deadletter) per process definition key.

Describe the solution you'd like

  • Add the processDefinitionKey query criteria to the BaseJobQuery interface and implement it with a join on act_re_procdef to existing Mybatis queries such as selectDeadLetterJobByQueryCriteriaSql

Describe alternatives you've considered

I have considered implementing a custom query from my app that would hit directly flowable engine data model, but I would prefer to keep my app decoupled from this Flowable internal DB model, using the standard Flowable Java API.

Additional context

Notes:

  • another feature request could handle the add of the processDefinitionKey criteria existing Jobs REST APIs
  • I have started to implement this change and plan to submit a PR
amporsim added a commit to amporsim/flowable-engine that referenced this issue Nov 24, 2023
amporsim added a commit to amporsim/flowable-engine that referenced this issue Nov 27, 2023
amporsim added a commit to amporsim/flowable-engine that referenced this issue Dec 1, 2023
amporsim added a commit to amporsim/flowable-engine that referenced this issue Dec 1, 2023
amporsim added a commit to amporsim/flowable-engine that referenced this issue Dec 1, 2023
amporsim added a commit to amporsim/flowable-engine that referenced this issue Dec 1, 2023
amporsim added a commit to amporsim/flowable-engine that referenced this issue Dec 1, 2023
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 a pull request may close this issue.

1 participant