Bug 1096128

Summary: Incremental compilation doesn't work with included KieBases
Product: [Retired] JBoss BRMS Platform 6 Reporter: Mario Fusco <mfusco>
Component: BREAssignee: Mario Fusco <mfusco>
Status: CLOSED CURRENTRELEASE QA Contact: Marek Winkler <mwinkler>
Severity: urgent Docs Contact:
Priority: urgent    
Version: 6.0.1CC: etirelli, mbaluch, rrajasek, rzhang
Target Milestone: CR1   
Target Release: 6.0.2   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-08-06 19:53:02 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 Mario Fusco 2014-05-09 09:20:05 UTC
KieContainer#updateToVersion does not update 'included' KieBases. For this reason also the KieScanner doesn't work as expected when a new version of a KieBase that is included in a second KieBase is available on the maven repository.

Comment 2 Mario Fusco 2014-05-09 09:22:31 UTC
Fixed by https://github.com/droolsjbpm/drools/commit/0b3233a3d

Comment 4 Mario Fusco 2014-05-12 10:34:36 UTC
Cherry-picked to 6.0.x branch with https://github.com/droolsjbpm/drools/commit/ebebeb504

Comment 5 Marek Winkler 2014-06-06 10:45:56 UTC
Verified that the changes in included KieBases are picked up, however, changes made in the KieModule, which includes the other KieModule, are not.

I have experienced the problem with business-central and KieScanner, but managed to isolate it in a unit test. Please see the reproducer in https://github.com/droolsjbpm/drools/pull/344.

Comment 6 Mario Fusco 2014-06-09 12:49:19 UTC
Fixed by https://github.com/droolsjbpm/drools/commit/7ac809256

Comment 7 Mario Fusco 2014-06-09 12:59:30 UTC
Cherry-picked to 6.1.x branch with https://github.com/droolsjbpm/drools/commit/3a059b98d

Comment 8 Mario Fusco 2014-06-09 13:06:58 UTC
Cherry-picked to 6.0.x branch with https://github.com/droolsjbpm/drools/commit/e788e74fa

Comment 9 Marek Winkler 2014-06-17 12:16:08 UTC
Verified on BRMS 6.0.2 CR1.