This is something I have been asking about for awhile, and am concerned about its
throughput implications. In EAP 5.x, I always remove the Valve from JBoss Web, remove the
interceptor from EJB 3 to get better performance. In the AS 7.x series, Jesper was good to
allow a configuration parameter to remove it from JCA (cannot be removed from JCA, but can
be put into non-debug mode).
So, I knew eventually this thing would pop back up, and the implementation approach is a
very big concern of mine, for performance. So, whatever is done there, it needs to be
removable, since this serves two purposes. One is just to catch bugs in the application,
and the other is the lazy enlistment of transactions. The later is only needed for
applications that, in my opinion is to support an anti-pattern that is codified in the
spec, and SPECjEnterprise2010 doesn't need that functionality (thank God!).
Andy
----- Original Message -----
From: "Carlo de Wolf" <cdewolf(a)redhat.com>
To: jpai(a)redhat.com
Cc: jboss-as7-dev(a)lists.jboss.org
Sent: Tuesday, October 18, 2011 10:51:24 AM
Subject: Re: [jboss-as7-dev] Better handling of transaction leaks?
Yes, I would even say that we need it in the thread pools
themselves.
Note that it does not have anything to do with cached connections.
Carlo
On 10/18/2011 05:58 PM, Jaikiran Pai wrote:
> While looking into a test failure I noticed that currently user
> applications can end up leaking transactions (associated with the
> invocation thread):
>
> Servlet {
>
> doGet() {
> UserTransaction.begin();
> doSomeOp();
> fail();
> // no tx rollback/commit
> }
> }
>
> Subsequent transactions on that thread can lead to failures and
> other
> sorts of issues. Looking back at previous versions of AS, we had a
> valve
>
http://anonsvn.jboss.org/repos/jbossas/trunk/tomcat/src/main/java/org/jbo...
> which used to cleanup (and log an error) about the leaking
> transactions.
>
> Do we want something similar for AS7? Also, are there other "entry
> points" (like servlets for web requests) where such leaks can
> happen and
> needs to be taken care off?
>
> -Jaikiran
>
> _______________________________________________
> jboss-as7-dev mailing list
> jboss-as7-dev(a)lists.jboss.org
>
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
_______________________________________________
jboss-as7-dev mailing list
jboss-as7-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev