Bug 1017578 - Simulation engine starts less instances than is set if the process contains a parallel gateway
Summary: Simulation engine starts less instances than is set if the process contains a...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss BPMS Platform 6
Classification: Retired
Component: jBPM Designer
Version: 6.0.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ER5
: 6.0.0
Assignee: Maciej Swiderski
QA Contact: Sona Mala
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-10-10 08:10 UTC by Sona Mala
Modified: 2014-08-06 20:07 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-08-06 20:07:35 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Video file - step to reproduce (6.05 MB, video/webm)
2013-10-10 08:10 UTC, Sona Mala
no flags Details
Process image (22.90 KB, image/png)
2013-10-10 08:11 UTC, Sona Mala
no flags Details
Actual result - Activity Instancies (43.88 KB, image/png)
2013-10-10 08:13 UTC, Sona Mala
no flags Details
Expected result - Activity Instancies (42.03 KB, image/png)
2013-10-10 08:14 UTC, Sona Mala
no flags Details
Actual result - Path 1 (57.35 KB, image/png)
2013-10-10 08:14 UTC, Sona Mala
no flags Details
Expected result - Path 1 (56.93 KB, image/png)
2013-10-10 08:15 UTC, Sona Mala
no flags Details
Server log (264.32 KB, image/png)
2013-10-10 08:20 UTC, Sona Mala
no flags Details

Description Sona Mala 2013-10-10 08:10:14 UTC
Created attachment 810361 [details]
Video file - step to reproduce

Description of problem:
If the process contains a parallel gateway with probabilities (x,y) of outgoing flows, which are set to x:y where x + y < 200 && x <= 100 && y <= 100, then the simulation engine starts less instances (for example 60:40 - starts 2 instances instead of 10).

If probabilities of outgoing flows are set to 100:100 then the number of started instances is right.


Version-Release number of selected component (if applicable):
6.0.0 BPMS ER3


Steps to Reproduce (see attached video):
1. create a process (see attached image of BPMN2)
2. set the probability of the flow from gateway to "h2" to 60
3. set the probability of the flow from gateway to "sss" to 40
4. run simulation 10 instances, 10 minutes

Actual results:
Number of instances for each activity is 2 (see attached screenshot)
Table for path:
------------------
|Path1       | 2 |
|Other Paths | 8 |
------------------

Expected results:
Number of instances for each activity is 10 (see attached screenshot)
Table for path:
-------------------
|Path1       | 10 |
|Other Paths |  0 |
-------------------

I believe that a probability parameter is useless for parallel gateway. I think that engine should ignore such parameters.


Additional info:

Comment 1 Sona Mala 2013-10-10 08:11:59 UTC
Created attachment 810362 [details]
Process image

Comment 2 Sona Mala 2013-10-10 08:13:37 UTC
Created attachment 810363 [details]
Actual result - Activity Instancies

Comment 3 Sona Mala 2013-10-10 08:14:07 UTC
Created attachment 810364 [details]
Expected result - Activity Instancies

Comment 4 Sona Mala 2013-10-10 08:14:42 UTC
Created attachment 810365 [details]
Actual result - Path 1

Comment 5 Sona Mala 2013-10-10 08:15:19 UTC
Created attachment 810366 [details]
Expected result - Path 1

Comment 6 Sona Mala 2013-10-10 08:20:35 UTC
Created attachment 810368 [details]
Server log

Comment 7 Maciej Swiderski 2013-10-14 17:48:13 UTC
roblem was caused by invalid handling of probability parameters for parallel gateway - they should be ignored as all outgoing sequence flows shall always be taken

droolsjbpm-integration
master
https://github.com/droolsjbpm/droolsjbpm-integration/commit/34c4feae7acd71c27ce67630fa7b091e7df977ff

6.0.x
https://github.com/droolsjbpm/droolsjbpm-integration/commit/5c20c8827e7b252034e499fb4e92803c978a74bb

Comment 8 Sona Mala 2013-12-12 13:24:05 UTC
Verified for ER5.

Comment 9 Sona Mala 2014-03-24 16:30:24 UTC
A test for this issue should be a part of certification matrix for Simulation features. Now, it does not exist.


Note You need to log in before you can comment on or make changes to this bug.