Executable flag in selectJobByQueryCriteriaSql

[prajwolbhandari1]

Environment (Required on creation)

reproduced on develop & 7.22

Description (Required on creation; please attach any relevant screenshots, stacktraces, log files, etc. to the ticket)

​​​​In job.xml it is implemented like this:

In docs:

Steps to reproduce (Required on creation)

job.xml -> selectJobByQueryCriteriaSql -> flag

camunda-bpm-platform/engine/src/main/resources/org/camunda/bpm/engine/impl/mapping/entity/Job.xml

Line 369 in 44544f2

<if test="executable">

Observed Behavior (Required on creation)

The executable filter excludes jobs tied to a suspended process instance but does not exclude suspended jobs. The documentation doesn't cover either scenario (look above as part of the screenshot).

Expected behavior (Required on creation)

It looks like it should exclude both process instances that are suspended (it already does) and also should exclude jobs that are suspended.

Solution Ideas

replace this:

and (RES.RETRIES_ > 0)
and (RES.DUEDATE_ is null or RES.DUEDATE_ <= #{now, jdbcType=TIMESTAMP})
and (
(RES.EXECUTION_ID_ is null)
or
(PI.SUSPENSION_STATE_ = 1 or PI.SUSPENSION_STATE_ is null)
)

to:

and (RES.RETRIES_ > 0) and (RES.DUEDATE_ is null or RES.DUEDATE_ <= #{now, jdbcType=TIMESTAMP}) and ( (RES.EXECUTION_ID_ is null) or (PI.SUSPENSION_STATE_ = 1 or PI.SUSPENSION_STATE_ is null) ) _and RES.SUSPENSION_STATE_ = 1_

Hints

Links

Breakdown

Pull Requests

Preview Give feedback

No tasks being tracked yet.

Dev2QA handover

• Does this ticket need a QA test and the testing goals are not clear from the description? Add a Dev2QA handover comment

[venetrius]

Hi @prajwolbhandari1,
Thank you for reporting this issue. I will look at it and get back to you.
If you have a bpmn that I can deploy and steps to reproduce the issue, that would be a great help.

Thanks,
Gergely

[prajwolbhandari1]

Hi @prajwolbhandari1, Thank you for reporting this issue. I will look at it and get back to you. If you have a bpmn that I can deploy and steps to reproduce the issue, that would be a great help.

Thanks, Gergely

@venetrius Sure. Thank you.

[prajwolbhandari1]

@venetrius any update?

[yanavasileva]

@prajwolbhandari1, could you please provide a process model and steps to reproduce to speed up the investigation of the ticket.