Re: [jta-dev] Handling java.lang.Error within the CDI interceptor

[Tom Jenkinson <tom.jenkinson@xxxxxxxxxx>]

Hi Ondra,

My understanding is runtime exceptions are unchecked exception and so in that sense are they effectively synonyms here.

I think the key thing is that neither @Transactional or the Synchronization notes explicitly state how to handle java.lang.Errors. I don't think the Jakarta EE platform itself does either. Looking at https://docs.oracle.com/javase/8/docs/api/java/lang/Error.html then I don't personally think that it's expected that the application deal with the error. I can't see reference to java.lang.Error in the platform specification either: https://jakarta.ee/specifications/platform/9/

So my assumption is it is intentional to not define how to deal with the abnormal condition as defined in the java.lang.Error javadoc in Jakarta EE project. That could be a useful topic for wider discussion in the Jakarta EE community if it's something the platform want constituent projects to deal with.

BTW - if I misunderstood a bit the question and you are looking to align the wording of RuntimeException and unchecked exception in the Jakarta Transactions specification for consistency in how that concept is referred to (and not asking about java.lang.Error) then I would be happy to review a PR aligning that wording.

Thanks for the question,

Tom

On Mon, 19 Apr 2021 at 12:29, Ondra Chaloupka <ochaloup@xxxxxxxxxx> wrote:

Hello,

I'm Ondra Chaloupka, a member of the Narayana team. Recently we've got a question from a user about handling of java.lang.Error thrown in a method of a CDI bean. The question is if a transaction started by the CDI @Transactional  interceptor should be marked for rollback.

As I was reading the current JTA specification it's not clear (at least to me). The spec says

By default checked exceptions do not result in the transactional interceptor marking the transaction for rollback and instances of RuntimeException and its subclasses do.

There is not said what to do with other un-checked exceptions. There is mentioned explicitly only the RuntimeTransaction here.

On the other hand, the text about Synchronization says that

An unchecked exception thrown by a registered Synchronization object causes the transaction to be aborted.

Should not be changed the text about CDI @Transactional interceptor in the same way - i.e. from explicit RuntimeException to term 'unchecked exception'?

Thank you,

Ondra

_______________________________________________
jta-dev mailing list
jta-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/jta-dev