Bug 1034094

Summary: Evaluation of the RHS in a rule with multiple matches using noloop and update gets cut short
Product: [Retired] JBoss BRMS Platform 6 Reporter: Greg Kable <gkable>
Component: BREAssignee: Mario Fusco <mfusco>
Status: CLOSED CURRENTRELEASE QA Contact: Radovan Synek <rsynek>
Severity: high Docs Contact:
Priority: urgent    
Version: 6.0.0CC: etirelli
Target Milestone: ER6   
Target Release: 6.0.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-08-06 20:18:12 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Some code to demonstrate the issue. none

Description Greg Kable 2013-11-25 09:14:01 UTC
Created attachment 828572 [details]
Some code to demonstrate the issue.

Description of problem:

If you have a rule with both noloop and update where there are multiple matches on the LHS, the RHS only runs for first match instead of running for all matches.  This is completely different to the behaviour under 5.3 and will break lots of existing code.


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


How reproducible:

Steps to Reproduce:
1. Build the attached code against BRMS6.0 and run - test will fail
2. Build the code against BRMS5.3 and the test will pass ...OR...
2. Or remove the uodate($order) call from the RHS under BRMS6.0 and the test will pass


Actual results:
Evaluation of the RHS stops after the update call


Expected results:
The RHS should be evaluated for all matches


Additional info:

Comment 4 Mario Fusco 2013-11-25 18:54:48 UTC
Fixed by https://github.com/droolsjbpm/drools/commit/a558125a2

Comment 5 Radovan Synek 2014-01-02 16:51:45 UTC
Verified on BRMS-6.0.0.ER7

Comment 6 Radovan Synek 2014-03-18 16:33:26 UTC
qe_test_coverage: covered by community unit tests