<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=us-ascii"><meta name=Generator content="Microsoft Word 14 (filtered medium)"><!--[if !mso]><style>v\:* {behavior:url(#default#VML);}
o\:* {behavior:url(#default#VML);}
w\:* {behavior:url(#default#VML);}
.shape {behavior:url(#default#VML);}
</style><![endif]--><style><!--
/* Font Definitions */
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:Tahoma;
panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
span.EmailStyle17
{mso-style-type:personal;
font-family:"Calibri","sans-serif";
color:windowtext;}
span.EmailStyle18
{mso-style-type:personal-reply;
font-family:"Calibri","sans-serif";
color:#1F497D;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]--></head><body lang=EN-US link=blue vlink=purple><div class=WordSection1><p class=MsoNormal><span style='color:#1F497D'>Anybody?<o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p><div style='border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt'><div><div style='border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm'><p class=MsoNormal><b><span style='font-size:10.0pt;font-family:"Tahoma","sans-serif"'>From:</span></b><span style='font-size:10.0pt;font-family:"Tahoma","sans-serif"'> rules-users-bounces@lists.jboss.org [mailto:rules-users-bounces@lists.jboss.org] <b>On Behalf Of </b>Swindells, Thomas<br><b>Sent:</b> 21 October 2011 14:49<br><b>To:</b> Rules Users List (rules-users@lists.jboss.org)<br><b>Subject:</b> [rules-users] WorkingMemoryLogger + statelessSessions = memory leak?<o:p></o:p></span></p></div></div><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Hi everyone,<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>I believe that there is a memory leak when using WorkingMemoryLoggers with statelessSessions. This was raised against 5.1.1 but I think it will still exist in the latest as well.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Our code basically has the following pattern:<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>KnowledgeBase kb ==;<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>while(nextEvent()) {<o:p></o:p></p><p class=MsoNormal> StatelessKnowledgeSession session = kb.newStatelessKnowledgeSession();<o:p></o:p></p><p class=MsoNormal> auditor = new WorkingMemoryInMemoryLogger(session); <o:p></o:p></p><p class=MsoNormal> session.execute(getEvent());<o:p></o:p></p><p class=MsoNormal> dumpAuditLog(auditor);<o:p></o:p></p><p class=MsoNormal> session = null; //no dispose it’s stateless<o:p></o:p></p><p class=MsoNormal>}<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>We were getting Out of Memory Exception on site and the stack dump showed a high number of MemoryLoggers on the heap.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Having a look at the code I think the problem is in the WorkingMemoryLogger constructor:<o:p></o:p></p><p class=MsoNormal>} else if (session instanceof StatelessKnowledgeSessionImpl) {<o:p></o:p></p><p class=MsoNormal> ((StatelessKnowledgeSessionImpl) session).workingMemoryEventSupport.addEventListener( this );<o:p></o:p></p><p class=MsoNormal> ((StatelessKnowledgeSessionImpl) session).agendaEventSupport.addEventListener( this );<o:p></o:p></p><p class=MsoNormal> ((StatelessKnowledgeSessionImpl) session).ruleFlowEventSupport.addEventListener( this );<o:p></o:p></p><p class=MsoNormal> ((StatelessKnowledgeSessionImpl) session).getRuleBase().addEventListener( this );<o:p></o:p></p><p class=MsoNormal> }<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>The last statement gets the knowledge base and attaches the event listener to it.<o:p></o:p></p><p class=MsoNormal>As far as I can see there is no way to dispose of the WorkingMemoryLogger and you don’t dispose stateless sessions (and it doesn’t know about the listener anyway). This means that even after the session is gone the rule base still has a reference to the logger.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>The work around is to make the logger a singleton and clear it in each loop rather than creating a new one each time, however this obviously doesn’t scale to multiple threads.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Have I missed some fundamental in the api or should I raise a Jira for this?<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Thomas<o:p></o:p></p><p class=MsoNormal><span style='font-size:12.0pt;font-family:"Times New Roman","serif"'><o:p> </o:p></span></p><div class=MsoNormal align=center style='text-align:center'><span style='font-size:12.0pt;font-family:"Times New Roman","serif"'><hr size=2 width="100%" align=center></span></div><p class=MsoNormal><span style='font-size:7.5pt;font-family:"Arial","sans-serif";color:gray'><br>**************************************************************************************<br>This message is confidential and intended only for the addressee. If you have received this message in error, please immediately notify the <a href="mailto:postmaster@nds.com">postmaster@nds.com</a> and delete it from your system as well as any copies. The content of e-mails as well as traffic data may be monitored by NDS for employment and security purposes. To protect the environment please do not print this e-mail unless necessary.<br><br>NDS Limited. Registered Office: One London Road, Staines, Middlesex, TW18 4EX, United Kingdom. A company registered in England and Wales. Registered no. 3080780. VAT no. GB 603 8808 40-00<br>**************************************************************************************</span><span style='font-size:12.0pt;font-family:"Times New Roman","serif"'><o:p></o:p></span></p></div></div></body></html>