Remove thenReturn that fetches unused values

Tussle0410 · rstoyanchev · commit 4f5e2eefe9bb · 2025-06-04T13:46:22.000+01:00
Closes gh-34883 Signed-off-by: ChanHyeongLee <cksgud410@gmail.com>
diff --git a/spring-r2dbc/src/main/java/org/springframework/r2dbc/connection/R2dbcTransactionManager.java b/spring-r2dbc/src/main/java/org/springframework/r2dbc/connection/R2dbcTransactionManager.java
@@ -219,7 +219,7 @@ protected Mono<Void> doBegin(TransactionSynchronizationManager synchronizationMa
 						if (txObject.isNewConnectionHolder()) {
 							synchronizationManager.bindResource(obtainConnectionFactory(), txObject.getConnectionHolder());
 						}
-					}).thenReturn(con).onErrorResume(ex -> {
+					}).onErrorResume(ex -> {
 						if (txObject.isNewConnectionHolder()) {
 							return ConnectionFactoryUtils.releaseConnection(con, obtainConnectionFactory())
 									.doOnTerminate(() -> txObject.setConnectionHolder(null, false))

Original file line number	Diff line number	Diff line change
`@@ -219,7 +219,7 @@ protected Mono<Void> doBegin(TransactionSynchronizationManager synchronizationMa`
`219`	`219`	`if (txObject.isNewConnectionHolder()) {`
`220`	`220`	`synchronizationManager.bindResource(obtainConnectionFactory(), txObject.getConnectionHolder());`
`221`	`221`	`}`
`222`		`- }).thenReturn(con).onErrorResume(ex -> {`
	`222`	`+ }).onErrorResume(ex -> {`
`223`	`223`	`if (txObject.isNewConnectionHolder()) {`
`224`	`224`	`return ConnectionFactoryUtils.releaseConnection(con, obtainConnectionFactory())`
`225`	`225`	`.doOnTerminate(() -> txObject.setConnectionHolder(null, false))`