Bug 1056599
| Summary: | Memory leak in phreak when a newly added fact is removed before a join node having that fact as right input is evaluated | ||
|---|---|---|---|
| Product: | [Retired] JBoss BRMS Platform 6 | Reporter: | Mario Fusco <mfusco> |
| Component: | BRE | Assignee: | Mario Fusco <mfusco> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Lukáš Petrovický <lpetrovi> |
| Severity: | urgent | Docs Contact: | |
| Priority: | urgent | ||
| Version: | 6.0.0 | CC: | etirelli, rrajasek |
| Target Milestone: | CR2 | ||
| 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:19:54 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: | |||
This was found today and being a memory leak, we would like to include the fix in 6.0.0.CR2. This fix has been cherry-picked to both 6.0.x and 6.0.0.GA.x branches. Mario Fusco <mario.fusco> updated the status of jira DROOLS-411 to Resolved The unit test to reproduce the issue has failed with CR2. Reopening. https://jenkins.mw.lab.eng.bos.redhat.com/hudson/view/BRMS/view/BRMS-6/view/BRMS-6-all/job/kie-unit-single/15/testReport/org.drools.compiler.integrationtests/Misc2Test/testStagedTupleLeak/ (In reply to Lukáš Petrovický from comment #6) > The unit test to reproduce the issue has failed with CR2. Reopening. The unit test DID NOT fail, mea culpa. This bug is indeed VERIFIED. |
When a fact that is the right input of a join node is added and then immediately removed before the join node is evaluated it gets added to the delete staged set of the right memory of that node. If the join node never gets evaluated the right tuple will remain forever in that staged set. Iterating this process with a big numbers of these facts can ultimately lead to a OutOfMemory. The following test case reproduces the problem. It doesn't make any assertion so it doesn't fail. Anyway putting a breakpoint in the RightTupleSetsImpl.addDelete method it is possible to see that the set has 100 RightTuple in its staged delete set. @Test public void testStagedTupleLeak() throws Exception { String str = "rule R1 when\n" + " $i : Integer()\n" + "then\n" + " insertLogical( $i.toString() );\n" + "end\n" + "\n" + "rule R2 when\n" + " $i : Integer()\n" + "then\n" + " delete( $i );\n" + "end\n" + "\n" + "rule R3 when\n" + " $l : Long()\n" + " $s : String( this == $l.toString() )\n" + "then\n" + "end\n"; KnowledgeBase kbase = loadKnowledgeBaseFromString(str); StatefulKnowledgeSession ksession = kbase.newStatefulKnowledgeSession(); for (int i = 0; i < 100; i++) { ksession.insert(i); ksession.fireAllRules(); } }