---

From: "Carlo de Wolf" <cdewolf@redhat.com>
To: jboss-as7-dev@lists.jboss.org
Sent: Wednesday, October 19, 2011 12:04:33 PM
Subject: Re: [jboss-as7-dev] Better handling of transaction leaks?

On 10/19/2011 07:34 PM, Remy Maucherat wrote:
> On Wed, 2011-10-19 at 19:29 +0200, Carlo de Wolf wrote:
>> On 10/19/2011 04:27 PM, Remy Maucherat wrote:
>>> On Wed, 2011-10-19 at 19:25 +0530, Jaikiran Pai wrote:
>>>> I agree that there should be a way to easily disable this. The only real
>>>> reason why I am in favour of this simple check/cleanup is to ensure that
>>>> some rogue application (which doesn't handle transactions correctly)
>>>> doesn't end up breaking/affecting some other (well-written) application
>>>> deployed on the same instance.
>>> A rogue application will be able to trivially do something that would
>>> affect other applications in the same instance, I believe.
>>>
>> EE.4.2.2 Transaction in Web Component Life Cycles
>> Returning from the service method to the network client with an active
>> transaction
>> context is an error. The web container is required to detect this error
>> and abort the
>> transaction.
>>
>> How do you propose we implement this?
> I didn't know it was a spec requirement :) Old legacy item, probably. So
> you win, congratulations, let's make our product worse.
>
No, we need our product to be the best. It just means we need to have
the option to break and bend compliance. Like Jaikiran says, if we can
have an easy setting to enable/disable this (preferably in
standalone.xml). We're good to go. I would even say to turn the check
off by default.

Carlo

That's what I have been saying too.  It needs to be there for the SPEC, but its really a development/debugging tool, and we can disable it for non-TCK runs (especially performance benchmarking, which is my main concern).  I already remove the current valve for all performance tests (EAP 5.x), since I get a good increase in throughput by doing so.

Andy

_______________________________________________
jboss-as7-dev mailing list
jboss-as7-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev