Return-Path: Delivered-To: apmail-geronimo-user-archive@www.apache.org Received: (qmail 34050 invoked from network); 28 Aug 2006 17:17:57 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 28 Aug 2006 17:17:57 -0000 Received: (qmail 75714 invoked by uid 500); 28 Aug 2006 17:17:56 -0000 Delivered-To: apmail-geronimo-user-archive@geronimo.apache.org Received: (qmail 75125 invoked by uid 500); 28 Aug 2006 17:17:55 -0000 Mailing-List: contact user-help@geronimo.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: List-Post: Reply-To: user@geronimo.apache.org List-Id: Delivered-To: mailing list user@geronimo.apache.org Received: (qmail 75114 invoked by uid 99); 28 Aug 2006 17:17:55 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 28 Aug 2006 10:17:55 -0700 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests=UNPARSEABLE_RELAY X-Spam-Check-By: apache.org Received-SPF: pass (asf.osuosl.org: local policy) Received: from [192.91.147.7] (HELO mailgw2a.lmco.com) (192.91.147.7) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 28 Aug 2006 10:17:53 -0700 Received: from emss03g01.ems.lmco.com (relay3.ems.lmco.com [141.240.4.144])by mailgw2a.lmco.com (LM-6) with ESMTP id k7SHHWMY004950for ; Mon, 28 Aug 2006 13:17:32 -0400 (EDT) Received: from CONVERSION-DAEMON.lmco.com by lmco.com (PMDF V6.3 #31239) id <0J4P00F01XD81Z@lmco.com> for user@geronimo.apache.org; Mon, 28 Aug 2006 13:17:32 -0400 (EDT) Received: from EMSS01I00.us.lmco.com ([129.197.181.70]) by lmco.com (PMDF V6.3 #31239) with ESMTP id <0J4P00G6LXD15W@lmco.com> for user@geronimo.apache.org; Mon, 28 Aug 2006 13:17:27 -0400 (EDT) Received: from EMSS01M10.us.lmco.com ([129.197.181.75]) by EMSS01I00.us.lmco.com with Microsoft SMTPSVC(5.0.2195.6713); Mon, 28 Aug 2006 10:17:26 -0700 Date: Mon, 28 Aug 2006 10:17:26 -0700 From: "Cameron, David A" Subject: RE: GeronimoTransactionManager IllegalStateException: Status is STATUS_NO_TRANSACTION To: user@geronimo.apache.org Message-id: MIME-version: 1.0 X-MIMEOLE: Produced By Microsoft Exchange V6.5 Content-type: text/plain; charset=iso-8859-1 Content-transfer-encoding: quoted-printable Thread-Topic: GeronimoTransactionManager IllegalStateException: Status is STATUS_NO_TRANSACTION Thread-Index: AcbFynCUsN93J9lxTxiP5ipGMUWeBAE+wavw Content-class: urn:content-classes:message X-MS-Has-Attach: X-MS-TNEF-Correlator: X-OriginalArrivalTime: 28 Aug 2006 17:17:26.0432 (UTC) FILETIME=[D5674200:01C6CAC5] X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N I posted an explanation of what I modified to get this working here: http://www.nabble.com/STATUS_NO_TRANSACTION-on-rollback--tf2124313.html=20 -----Original Message----- From: news [mailto:news@sea.gmane.org] On Behalf Of Joerg Heinicke Sent: Tuesday, August 22, 2006 2:07 AM To: user@geronimo.apache.org Subject: Re: GeronimoTransactionManager IllegalStateException: Status is = STATUS_NO_TRANSACTION Cameron, David A writes: > My gut says to fix it for the scenario and not remove something that=20 > may have a purpose. What I would do is add a check to verify that the = > TransactionImpl is not in STATUS_NO_TRANSACTION status before trying=20 > to roll it back. It has other, similar checks already. Just not that = > one. Yes, this is something I came across as well. InheritableTransactionContext.rollbackAndThrow() has a check for = STATUS_NO_TRANSACTION for the rollback() on the TransactionManager, but = not for the second rollback() directly on the Transaction instance. J=F6rg