While migrating from Hibernate 4.x to latest Hibernate 5 version, I'm encountering an issue with regards to transaction management. In our code (actually a unit test), there is a transaction manager that begins a JTA transaction, followed by a sequence of multiple invocations of a method that call to creates a Session to do a save (in which a beginTransaction() is done). Below is an example that reproduces the issue (the scenario is not using Spring or any other container managed transaction management):
transactionManager.begin();
saveOrUpdate(entity1);
saveOrUpdate(entity2);
...
transactionManager.commit();
private void saveOrUpdate(SomeEntity entity) {
try (Session session = sessionFactory.openSession()) {
session.setFlushMode(FlushMode.AUTO);
session.beginTransaction(); try {
session.saveOrUpdate(entity);
session.getTransaction().commit();
} catch (Exception ex) {
session.getTransaction().rollback();
throw RuntimeException(ex);
}
}
}
This is causing an IllegalStateException to be thrown with the message "Transaction already active". This behavior seems to have been introduced in Hibernate 5.2.0 (this is the commit) to comply with JPA's EntityTransaction. Previously, Hibernate just ignored the beginning the physical transaction itself because it knows an enclosing transaction is present: it just creates a wrapper JtaTransaction with isInitiator set to false. This exception is thrown in org.hibernate.engine.transaction.internal.TransactionImpl, specifically the begin() method:
@Override
public void begin() {
if ( !session.isOpen() ) {
throw new IllegalStateException( "Cannot begin Transaction on closed Session/EntityManager" );
}
if ( transactionDriverControl == null ) {
transactionDriverControl = transactionCoordinator.getTransactionDriverControl();
}
if ( isActive() ) { throw new IllegalStateException( "Transaction already active" );
}
LOG.debug( "begin" );
this.transactionDriverControl.begin();
}
This contradicts with the user manual, where it says the below:
/ Note: depending on the JtaPlatform used and some optional settings,
Session session = sessionFactory.openSession();
try {
session.getTransaction().begin();
Is there a way to restore the old behavior? Our code relies on such a custom TM scenario. It would be great to have your feedback on this. |