[jboss-jira] [JBoss JIRA] (JBJCA-1159) ConnectionListener leaked if TSR throws IllegalStateException

[RH Bugzilla Integration (JIRA)]

    [ https://issues.jboss.org/browse/JBJCA-1159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12989073#comment-12989073 ] 

RH Bugzilla Integration commented on JBJCA-1159:
------------------------------------------------

Vaclav Tunka <vtunka at redhat.com> changed the Status of [bug 1081058|https://bugzilla.redhat.com/show_bug.cgi?id=1081058] from ASSIGNED to ON_QA

> ConnectionListener leaked if TSR throws IllegalStateException
> -------------------------------------------------------------
>
>                 Key: JBJCA-1159
>                 URL: https://issues.jboss.org/browse/JBJCA-1159
>             Project: IronJacamar
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: Core
>    Affects Versions: 1.0.24.Final, 1.1.4.Final
>         Environment: AS7 EAP6.1
>            Reporter: Koen Janssens
>            Assignee: Jesper Pedersen
>            Priority: Blocker
>             Fix For: 1.0.25.Final, 1.1.5.Final, 1.2.0.Beta1
>
>
> When  a connection is retrieved from the MCP, ironjacamar will register it to ongoing JTA transaction. However, if the ongoing TX is not 'active' anymore the connection is lost. 
> The code of AbstractPool demonstrates the problem. The call to getLock will throw an exception if the current TX is not active anymore and the cl is not returned to the pool.
> This issue can be reproduced on AS7 by using any EJB that requires a tx and does something with a DB connection. Put a breakpoint in the code below after retrieving the connectionlistener, and then wait for the transaction to timeout. Once that's done, continue the thread. The connection is not release (can be seen in JMX)
> We have noticed this problem regularly during our performance tests. 
> {code}
>  ConnectionListener cl = mcp.getConnection(subject, cri);
>       if (trace)
>          log.tracef("Got connection from pool tracked by transaction=%s tx=%s", cl, trackByTransaction);
>       TransactionSynchronizationRegistry tsr = getTransactionSynchronizationRegistry();
>       Lock lock = getLock();
>       try
>       {
>          lock.lockInterruptibly();
>       }
>       catch (InterruptedException ie)
>       {
>          Thread.interrupted();
>          throw new ResourceException(bundle.unableObtainLock(), ie);
>       }
> {code}
> It seems this issue was introduced by changes done for https://issues.jboss.org/browse/JBJCA-572



--
This message was sent by Atlassian JIRA
(v6.2.6#6264)