[jboss-as7-dev] Better handling of transaction leaks?

[Andrig Miller]

----- Original Message -----

> From: "Carlo de Wolf" <cdewolf at redhat.com>
> To: jboss-as7-dev at 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 at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/jboss-as7-dev/attachments/20111019/4658339a/attachment-0001.html