[AMQ-6391] Memory leak with FailoverTransport when sending TX messages from MDB

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Not A Problem
• Assignee: Unassigned
• Reporter: Patrik Dudits
• Created: 09/Aug/16 13:06
• Updated: 17/Oct/23 09:40
• Resolved: 17/Oct/23 09:40

Description

We observe memory leak in FailoverTransport.stateTracker.connectionStates.transactions when using XA Transactions in activemq-rar, sending message within same transaction and not using useInboundSession.

In such constellation there are two connections enlisted within same transaction. During commit the transaction manager will execute commit on one of the resources, per JTA 1.2 section 3.3.1 ("(TransactionManager) ensures that the same resource manager only receives one set of prepare-commit calls for completing the target global transaction ".) TransactionContext will propagate the afterCommit to all contexts participating in same transaction. However, this is not enough for ConnectionStateTracker, which only reacts to TransactionInfo command. In effect, when two connection are enlisted in same transaction, just commands of one of them is cleared upon commit, leading to memory leak.

Since I presume the TransactionInfo should be sent only once for commit of single transaction, ConnectionStateTracker needs to clear state for the acknowledged transactions regardless of connection id in the transaction command.

Issue Links

• links to: GitHub Pull Request #197