feat: upgrade to R2DBC SPI [do not merge] #697

zhumin8 · 2023-06-08T15:50:55Z

Attempt to upgrade to R2DBC SPI 1.0 #487

This is a draft PR intent to estimate upgrade effort.

Change in r2dbc:
io.r2dbc.spi.Result$getRowsUpdated() now returns Publisher<Long> instead of Publisher<Integer>
r2dbc/r2dbc-spi@03977f5, r2dbc/r2dbc-spi#255

Fixed compiling errors
Fixed unit test errors
integration test errors caused by return type change Integer -> Long

error log

[ERROR] Failures: 
[ERROR]   ClientLibraryBasedIntegrationTest.insertMultipleBoundParameterSetsInTransaction expectation "expectNext(1)" failed (expected value: 1; actual value: 1)
[ERROR]   ClientLibraryBasedIntegrationTest.selectQueryReturnsUpdatedDataDuringAndAfterTransactionCommit expectation "expectNext(1)" failed (expected value: 1; actual value: 1)
[ERROR]   ClientLibraryBasedIntegrationTest.selectQueryReturnsUpdatedDataDuringTransactionButNotAfterTransactionRollback expectation "expectNext(1)" failed (expected value: 1; actual value: 1)
[ERROR]   ClientLibraryBasedIntegrationTest.testStaleRead expectation "row inserted" failed (expected value: 1; actual value: 1)
[ERROR]   ClientLibraryBasedIntegrationTest.testStrongReadFromSubclassedConnection expectation "row inserted" failed (expected value: 1; actual value: 1)
[ERROR]   ClientLibraryBasedIntegrationTest.testTransactionFollowedByStandaloneStatementCommitted expectation "expectNext(1)" failed (expected value: 1; actual value: 1)
[ERROR]   ClientLibraryBasedIntegrationTest.testTransactionMultipleStatementsCommitted expectation "expectNext(1)" failed (expected value: 1; actual value: 1)
[ERROR]   ClientLibraryBasedIntegrationTest.testTransactionRolledBack expectation "expectNext(1)" failed (expected value: 1; actual value: 1)
[ERROR]   ClientLibraryBasedIntegrationTest.testTransactionSingleStatementCommitted expectation "expectNext(1)" failed (expected value: 1; actual value: 1)
[INFO] 
[ERROR] Tests run: 60, Failures: 9, Errors: 0, Skipped: 34