[hibernate-issues] [Hibernate-JIRA] Commented: (HHH-6599) Hibernate mark as rollback even for no transaction requests

Hibernate mark as rollback even for no transaction requests ----------------------------------------------------------- Key: HHH-6599 URL: http://opensource.atlassian.com/projects/hibernate/browse/HHH-6599 Project: Hibernate Core Issue Type: Bug Components: entity-manager Affects Versions: 3.6.7 Reporter: Alessandro Lazarotti When used a Hibernate EntityManager in a CMT Bean and a GenericJDBCException is launched, Hibernate will try a rollback even if there is not an active transaction, generating the exception: <org.hibernate.ejb.AbstractEntityManagerImpl> Unable to mark for rollback on PersistenceException: java.lang.IllegalStateException: [com.arjuna.ats.internal.jta.transaction.arjunacore.nosuchtx] [com.arjuna.ats.internal.jta.transaction.arjunacore.nosuchtx] No such transaction! at com.arjuna.ats.internal.jta.transaction.arjunacore.BaseTransaction.setRollbackOnly(BaseTransaction.java:188) at com.arjuna.ats.jbossatx.BaseTransactionManagerDelegate.setRollbackOnly(BaseTransactionManagerDelegate.java:135) at org.hibernate.ejb.AbstractEntityManagerImpl.markAsRollback(AbstractEntityManagerImpl.java:440) at org.hibernate.ejb.AbstractEntityManagerImpl.throwPersistenceException(AbstractEntityManagerImpl.java:595) at org.hibernate.ejb.AbstractEntityManagerImpl.throwPersistenceException(AbstractEntityManagerImpl.java:637) at org.hibernate.ejb.QueryImpl.getResultList(QueryImpl.java:74) ... hiding the original "Caused By" from the other Exception (GenericJDBCException). As example, a case where some error in runtime happens in a database procedure execution and the bean method is annotated as "NOT_SUPPORTED" transaction, this behaviour will happen and the original exception will be lost, the root exception is overridden by "No such transaction" exception and so the cause is not reported in log files. Hibernate should not try a rollback if there is not a transaction. Fixing it is necessary update the method: "protected void markAsRollback()" .. from org.hibernate.ejb.AbstractEntityManagerImpl.java That method just check no CMT JTA transaction: "if ( tx.isActive() ) { tx.setRollbackOnly(); }" The same check is necessary for CMT JTA using transactionManager.getStatus and looking if it is different than Status.NoTransaction. This check should be done above the code that call transactionManager.setRollbackOnly();