Exceptions thrown in Interceptor.beforeTransactionCompletion should abort the commit

Description

Currently SessionImpl.beforeTransactionCompletion.beforeTransactionCompletion() will catch, log and ignore any exception thrown by the Interceptor beforeTransactionCompletion(). This means that if there is a problem in the interceptor, the commit will still be carried out.

I can find no documentation or motivation for this behavior. It should be changed so that exceptions will abort the commit and allow the code to do rollback instead.

Environment

None

Assignee

Brett Meyer

Reporter

Oskar Berggren

Fix versions

None

Labels

backPortable

None

Suitable for new contributors

None

Requires Release Note

None

Pull Request

None

backportDecision

None

Components

Affects versions

Priority

Major
Configure