[hibernate-commits] [hibernate/hibernate-orm] ec6cd5: HHH-15082 Abort JDBC batches on runtime exceptions...
Branch: refs/heads/main
Home: https://github.com/hibernate/hibernate-orm
Commit: ec6cd5d0cc1cc0cb9f4e9c8616ce318eb34ebed1
https://github.com/hibernate/hibernate-orm/commit/ec6cd5d0cc1cc0cb9f4e9c8...
Author: Yoann Rodière <yoann(a)hibernate.org>
Date: 2022-02-18 (Fri, 18 Feb 2022)
Changed paths:
M
hibernate-core/src/main/java/org/hibernate/engine/jdbc/batch/internal/BatchingBatch.java
M
hibernate-core/src/main/java/org/hibernate/engine/jdbc/batch/internal/NonBatchingBatch.java
M
hibernate-core/src/main/java/org/hibernate/persister/entity/AbstractEntityPersister.java
Log Message:
-----------
HHH-15082 Abort JDBC batches on runtime exceptions as well as SQLException
Commit: 14c8fb752d27b1e8a4e70a7543857e806105b2c0
https://github.com/hibernate/hibernate-orm/commit/14c8fb752d27b1e8a4e70a7...
Author: Yoann Rodière <yoann(a)hibernate.org>
Date: 2022-02-18 (Fri, 18 Feb 2022)
Changed paths:
A
hibernate-core/src/test/java/org/hibernate/orm/test/jpa/transaction/batch/FailingAddToBatchTest.java
Log Message:
-----------
HHH-15082 Test that batch statements are aborted if a RuntimeException is thrown by
Batch#addToBatch
Such an exception can be thrown if an expectation
(org.hibernate.jdbc.Expectation) is not met, for example if an update
statement for a given entity affects 0 rows (e.g. because of a concurrent
update).
Commit: cf9d131d355f3505852aab9b3539bc00d6501167
https://github.com/hibernate/hibernate-orm/commit/cf9d131d355f3505852aab9...
Author: Yoann Rodière <yoann(a)hibernate.org>
Date: 2022-02-18 (Fri, 18 Feb 2022)
Changed paths:
M
hibernate-core/src/main/java/org/hibernate/engine/jdbc/batch/internal/AbstractBatchImpl.java
M
hibernate-core/src/main/java/org/hibernate/engine/jdbc/batch/internal/BatchingBatch.java
M
hibernate-core/src/main/java/org/hibernate/engine/jdbc/batch/internal/NonBatchingBatch.java
Log Message:
-----------
HHH-15082 Correctly propagate the original exception when aborting a JDBC batch fails
Not strictly necessary, but it's related to these changes
and I think it's a good idea.
Compare: https://github.com/hibernate/hibernate-orm/compare/73e9859feab7...cf9d131...