Bug 1273087

Summary: Errors using incremental update of the KieBase and KieSession
Product: [Retired] JBoss BRMS Platform 6 Reporter: Martin Weiler <mweiler>
Component: BREAssignee: Mario Fusco <mfusco>
Status: CLOSED EOL QA Contact: Marek Winkler <mwinkler>
Severity: urgent Docs Contact:
Priority: urgent    
Version: 6.1.0CC: kverlaen
Target Milestone: CR2   
Target Release: 6.2.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-03-27 19:03:45 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:

Description Martin Weiler 2015-10-19 14:25:49 UTC 
Description of problem:
Platform issue to ensure the fixes for the following two JIRAs make it into BRMS 6.2.0:

* Problem when a rule removal causes the merge of 2 segments and the first of the 2 haven't been initialized yet
  https://issues.jboss.org/browse/DROOLS-950

* Removal of a rule having a sub-sub-network doesn't cleanup the outermost subnetwork
  https://issues.jboss.org/browse/DROOLS-954


Additional info:
See related JIRAs for further info
Comment 1 JBoss JIRA Server 2015-10-19 16:09:03 UTC 
Mario Fusco <mario.fusco> updated the status of jira DROOLS-954 to Closed
Comment 2 Mario Fusco 2015-10-19 17:13:36 UTC 
I added some other incremental compilation issues that have been fixed of them to keep track of all of them with a single ticket.

All these fixes have been already backported to 6.3.x branch (BRMS 6.2) with the following commits:

DROOLS-801: https://github.com/droolsjbpm/drools/commit/350b8f3c4313cffbfed5145c9b5920ceaace275e

DROOLS-891: https://github.com/droolsjbpm/drools/commit/88e00ec8d575d7bf7aae1307a00fa20b015f058b

DROOLS-893: https://github.com/droolsjbpm/drools/commit/aff306395030bfb9924b149d143d7591ce2be438

DROOLS-930: https://github.com/droolsjbpm/drools/commit/25ae132d6522aa6e022eeea36df83b3e4e59dc65 + https://github.com/droolsjbpm/drools/commit/e6b93fff9a071deb4b5d2b9c61ee791e41062bf7

DROOLS-950: https://github.com/droolsjbpm/drools/commit/1916738fb3b648a57f504b0e43b9c3a80d8c3d1d

DROOLS-951: https://github.com/droolsjbpm/drools/commit/74640f6a123d47fa33e122de3d331b544262c602

DROOLS-954: https://github.com/droolsjbpm/drools/commit/8c1c79b59577bc64085a62ae85037b8fd29e42b0
Comment 3 Marek Winkler 2015-11-05 13:13:26 UTC 
The tests have passed and no regression has been found. However, there is a minor problem with missing @Test annotation in [1] causing one test not to run.

Please see PR [2] adding the missing annotation.

I have run the test locally and it passed; however, I am setting this BZ to assigned to get the test fixed in the product branch as well.

[1] https://github.com/droolsjbpm/drools/blob/74640f6a123d47fa33e122de3d331b544262c602/drools-compiler/src/test/java/org/drools/compiler/integrationtests/IncrementalCompilationTest.java#L2084

[2] https://github.com/droolsjbpm/drools/pull/546
Comment 4 Mario Fusco 2015-11-05 13:50:42 UTC 
Fixed by https://github.com/droolsjbpm/drools/commit/a53b630f21ff8ab65de8f22d28611160ba5936d3
Comment 6 Mario Fusco 2015-11-19 15:58:04 UTC 
Last commit cherry-picked by https://github.com/droolsjbpm/drools/commit/83a8cec27
Comment 8 Marek Winkler 2015-12-02 13:02:08 UTC 
Verified in BRMS 6.2.0 CR2.