Patches item #684262, was opened at 2003-02-10 17:14
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=376687&aid=684262&group_id=22866
Category: JBossCX
Group: v3.0 Rabbit Hole
Status: Open
Resolution: None
Priority: 5
Submitted By: David Croley (dcroley)
Assigned to: Nobody/Anonymous (nobody)
Summary: ConnectionManager does not release connections if excepted
Initial Comment:
In the ConnectionManager (under 3.0.4), we have been
doing some integration with Toplink and I think that I
have found a bug in the JBoss code. The problem we are
seeing is that managed connections are not be released
when they should be.
In LocalTxConnectionManager.java, there is some code in
connectionClosed() method that looks like:
if (isManagedConnectionFree())
{
log.info("called unregisterAssociation, delisting");
//no more handles
delist();
log.info("called unregisterAssociation, returning");
returnManagedConnection(this.getManagedConnection(),
false);
}
The problem is that if there is an exception in the
delist() function call, the managed connection is never
released. This happens silently as the code only
catches ResourceExceptions, but some parent is catching
the exception that we are getting. By placing a
catch-all try clause around the delist(), I see that we
get...
java.lang.IllegalStateException: Already committed.
in some cases. We have fixed the problem for us by
simply reordering the statements so that the delist()
happens after the call to "returnManagedConnection()",
but a better solution might be to use a better
try-catch-finally clause.
Looks like this code has been rewritten in the 4.0RC's,
but it would be nice to fix this for the 3.x branch.
David Croley
[EMAIL PROTECTED]
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=376687&aid=684262&group_id=22866
-------------------------------------------------------
This SF.NET email is sponsored by:
SourceForge Enterprise Edition + IBM + LinuxWorld = Something 2 See!
http://www.vasoftware.com
_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development
