[jbossts-issues] [JBoss JIRA] Commented: (JBTM-444) Slow memory leak in BaseTransaction

Mark Little (JIRA) jira-events at lists.jboss.org
Thu Jan 8 10:07:04 EST 2009

    [ https://jira.jboss.org/jira/browse/JBTM-444?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12445872#action_12445872 ] 

Mark Little commented on JBTM-444:

Should clarify: the JTS version uses ThreadLocal in Current, which is what BaseTransaction relies on in jtax.

> Slow memory leak in BaseTransaction
> -----------------------------------
>                 Key: JBTM-444
>                 URL: https://jira.jboss.org/jira/browse/JBTM-444
>             Project: JBoss Transaction Manager
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: JTA Implementation
>    Affects Versions: 4.2.3.SP5, 4.2.3.SP6, 4.2.3.SP7
>         Environment: Windows XP and Solaris, Java 1.5.0_14, JBoss 4.2.2 (so with JBossTS 4.2.3_SP6, but the problem is still there in SP7)
>            Reporter: Andrew McDowell
>            Assignee: Mark Little
>             Fix For: 4.6
>         Attachments: JBTM-444.patch, mat-screenshot-1.jpg, matscreenshot.png
>  We had  JBossAS running for approximately 4 weeks under extremely heavy load, and reached an OutOfMemory condition.  Heap Dump analysis showed that the class com.arjuna.ats.internal.jta.transaction.arjunacore.BaseTransaction dominated 85% of the heap space.
> In the class com.arjuna.ats.internal.jta.transaction.arjunacore.BaseTransaction, there is a private Hashtable _timeouts, into which a thread id and a timeout value are placed when transactions are started.  However, the entries in the hashtable are never removed, even if the thread dies.   Given all the JBossAS invokers for EJBs create/destroy worker threads,  it means that client transactions can leak approximately 600 bytes (the String, Integer and Hashtable$Entry).
> The _timeouts table in our heap dump had some 6 million entries and a retained heap size of over 1Gb.
> Our workaround to the problem is that we are actively monitoring heap size and performing rolling restarts when the heap space is too low.
> You may want to consider using a ThreadLocal to store the timeout instead of a Hashtable, which would accomplish the goal of storing the timeout value per thread, but would be collected if the Thread died, but I will leave implementation speculation to you unless I find the time to submit a patch.

This message is automatically generated by JIRA.
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira


More information about the jbossts-issues mailing list