[#2535] Fix operations order in BatchingConnection

DavideD · DavideD · commit dc1ff79026c9 · 2025-09-26T18:02:53.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
@@ -106,9 +106,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
`@@ -106,9 +106,8 @@ public CompletionStage<Void> update(`
`106`	`106`	`return voidFuture();`
`107`	`107`	`}`
`108`	`108`	`else {`
`109`		`- CompletionStage<Void> lastBatch = executeBatch();`
`110`		`- newBatch( sql, paramValues, expectation );`
`111`		`- return lastBatch;`
	`109`	`+ return executeBatch()`
	`110`	`+ .thenAccept( v -> newBatch( sql, paramValues, expectation ) );`
`112`	`111`	`}`
`113`	`112`	`}`
`114`	`113`	`}`