[#2535] Fix operations order in BatchingConnection

DavideD · DavideD · commit ea587e73530c · 2025-09-26T19:44:03.000+02:00
diff --git a/hibernate-reactive-core/src/main/java/org/hibernate/reactive/pool/BatchingConnection.java b/hibernate-reactive-core/src/main/java/org/hibernate/reactive/pool/BatchingConnection.java
@@ -104,9 +104,8 @@ public CompletionStage<Void> update(
 					return voidFuture();
 				}
 				else {
-					CompletionStage<Void> lastBatch = executeBatch();
-					newBatch( sql, paramValues, expectation );
-					return lastBatch;
+					return executeBatch()
+							.thenAccept( v -> newBatch( sql, paramValues, expectation ) );
 				}
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -104,9 +104,8 @@ public CompletionStage<Void> update(`
`104`	`104`	`return voidFuture();`
`105`	`105`	`}`
`106`	`106`	`else {`
`107`		`- CompletionStage<Void> lastBatch = executeBatch();`
`108`		`- newBatch( sql, paramValues, expectation );`
`109`		`- return lastBatch;`
	`107`	`+ return executeBatch()`
	`108`	`+ .thenAccept( v -> newBatch( sql, paramValues, expectation ) );`
`110`	`109`	`}`
`111`	`110`	`}`
`112`	`111`	`}`