[prev in list] [next in list] [prev in thread] [next in thread] 

List:       activemq-dev
Subject:    [jira] Commented: (AMQ-2556) Memory leaks with XA Transactions
From:       "Marc Layer (JIRA)" <jira () apache ! org>
Date:       2010-05-27 13:24:54
Message-ID: 25183983.4651274966694475.JavaMail.jira () thor
[Download RAW message or body]


    [ https://issues.apache.org/activemq/browse/AMQ-2556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=59572#action_59572 \
] 

Marc Layer commented on AMQ-2556:
---------------------------------

Even with the fix (r930135), is it not still possible that {{TransactionContexts}} \
remain indefinitely in {{TransactionContext.ENDED_XA_TRANSACTION_CONTEXTS}}? \
{{syncSendPacketWithInterruptionHandling}} could throw a {{JMSException}} before the \
context is removed from the Map, if I am not mistaken.

> Memory leaks with XA Transactions (case of read-only transactions)
> ------------------------------------------------------------------
> 
> Key: AMQ-2556
> URL: https://issues.apache.org/activemq/browse/AMQ-2556
> Project: ActiveMQ
> Issue Type: Bug
> Components: Broker, JMS client
> Affects Versions: 5.1.0, 5.2.0, 5.3.0
> Environment: ActiveMQ 5.3.0
> Atomikos Transactions Essentials 3.5.8
> Spring 2.5.6
> Oracle 11g (thin driver version 11.1.0.7.0)
> Reporter: Daniel Santos
> Assignee: Gary Tully
> Fix For: 5.4.0
> 
> Attachments: XAConsumer.zip
> 
> 
> Some transaction related information are not released in case of read-only \
> transactions (where not messages have been consumed or produced) by broker and \
> client, that causes an OutOfMemory after running some time. Fields that hold these \
>                 "transaction related information" are (at least) :
> - {{org.apache.activemq.state.ConnectionState.transactions}}
> - {{org.apache.activemq.TransactionContext.ENDED_XA_TRANSACTION_CONTEXTS}}
> After serach, it seems that the case of read-only XA transactions (that terminates \
>                 at prepare time) has been missed in some code like :
> - {{org.apache.activemq.broker.TransportConnection}} that puts TransactionState in \
> {{org.apache.activemq.state.ConnectionState.transactions}} at the beginning, \
> release them at commit (or rollback) time *but not at prepare time where result is \
>                 {{XAResource.XA_RDONLY}}*
> - {{org.apache.activemq.TransactionContext}} that do the same mistake via \
> ENDED_XA_TRANSACTION_CONTEXTS in prepare() _Note that the case of read-only \
> transactions seems correctly done by \
> {{org.apache.activemq.transaction.XATransaction}} (very interesting comment here \
> http://fisheye6.atlassian.com/browse/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/transaction/XATransaction.java?r=809940#l175)_
> 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic