Mailing-List: contact open-jpa-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: open-jpa-dev@incubator.apache.org
Received-SPF: pass (herse.apache.org: domain of plinskey@bea.com designates
 66.248.192.22 as permitted sender)
Content-class: urn:content-classes:message
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Subject: RE: Exceptions thrown from callbacks
Date: Thu, 1 Feb 2007 16:02:14 -0800
Message-ID: <7D856CDFE035FF45A0420ACBD71BDD63031EA51D@repbex02.amer.bea.com>
In-Reply-To: <5FAB46C0-3476-4D36-B63B-D14EC1A4B7AA@iq80.com>
Thread-Topic: Exceptions thrown from callbacks
Thread-Index: AcdGXJZNkAVsCsiETgqHGxOAsPdRqwAAIzfQ
References: <7D856CDFE035FF45A0420ACBD71BDD63031EA4EB@repbex02.amer.bea.com>
 <5FAB46C0-3476-4D36-B63B-D14EC1A4B7AA@iq80.com>
From: "Patrick Linskey" <plinskey@bea.com>
To: <open-jpa-dev@incubator.apache.org>

> > 3.5.6:
> >
> > "Lifecycle callback methods may throw runtime exceptions. A runtime
> > exception thrown by a callback method that executes within a=20=20
> > transaction
> > causes that transaction to be rolled back. No further lifecycle=20=20
> > callback
> > methods will be invoked after a runtime exception is thrown."
>=20
> One other thing to check is if the tx is marked rollback=20
> only, should=20=20
> we not call anymore lifecycle events or is that "NoFurther lifecycle=20=
=20
> callback" clause supposed to apply to the single entity that failed.

You mean: if someone calls setRollbackOnly() and then calls flush(),
what should we do about the @PreUpdate callback? I don't have a strong
opinion either way. My read of the spec is that we should execute the
callback; callbacks should only be aborted if the exception is thrown
during a time when the callbacks would have otherwise been invoked.

> > Further, I think that 3.7 trumps 3.5.6, so if an exception is thrown
> > from a callback during a find(), we should be marking the=20
> transaction
> > for rollback, rather than actually rolling it back.
>=20
> That is standard EJB behavior and what I would expect from the EJB=20=20
> spec committee.

Agreed.

-Patrick
_______________________________________________________________________
Notice:  This email message, together with any attachments, may contain
information  of  BEA Systems,  Inc.,  its subsidiaries  and  affiliated
entities,  that may be confidential,  proprietary,  copyrighted  and/or
legally privileged, and is intended solely for the use of the individual
or entity named in this message. If you are not the intended recipient,
and have received this message in error, please immediately return this
by email and then delete it.