HHH-9980 - Implement mark-for-rollback-only handling for JdbcResourceLocalTransactionCoordinatorImpl

Author: sebersole

hibernate-core/src/main/java/org/hibernate/resource/transaction/backend/jdbc/internal/JdbcResourceLocalTransactionCoordinatorImpl.java

@@ -230,7 +230,7 @@ public void rollback() {
 
 		@Override
 		public TransactionStatus getStatus() {
-			return jdbcResourceTransaction.getStatus();
+			return rollbackOnly ? TransactionStatus.MARKED_ROLLBACK : jdbcResourceTransaction.getStatus();
 		}
 
 		@Override

hibernate-core/src/test/java/org/hibernate/test/resource/transaction/jdbc/BasicJdbcTransactionTests.java

@@ -10,6 +10,7 @@
 import org.hibernate.resource.transaction.TransactionCoordinator;
 import org.hibernate.resource.transaction.TransactionCoordinatorBuilder;
 import org.hibernate.resource.transaction.backend.jdbc.internal.JdbcResourceLocalTransactionCoordinatorBuilderImpl;
+import org.hibernate.resource.transaction.spi.TransactionStatus;
 
 import org.hibernate.test.resource.common.SynchronizationCollectorImpl;
 import org.junit.Test;
@@ -69,14 +70,21 @@ public boolean shouldAutoJoinTransaction() {
 				}
 		);
 
+		assertEquals( TransactionStatus.NOT_ACTIVE, transactionCoordinator.getTransactionDriverControl().getStatus() );
+
 		transactionCoordinator.getTransactionDriverControl().begin();
+		assertEquals( TransactionStatus.ACTIVE, transactionCoordinator.getTransactionDriverControl().getStatus() );
+
 		transactionCoordinator.getTransactionDriverControl().markRollbackOnly();
+		assertEquals( TransactionStatus.MARKED_ROLLBACK, transactionCoordinator.getTransactionDriverControl().getStatus() );
+
 		try {
 			transactionCoordinator.getTransactionDriverControl().commit();
 		}
 		catch (TransactionException expected) {
 		}
 		finally {
+			assertEquals( TransactionStatus.MARKED_ROLLBACK, transactionCoordinator.getTransactionDriverControl().getStatus() );
 			transactionCoordinator.getTransactionDriverControl().rollback();
 		}
 	}