r2dbc / r2dbc-mssql

R2DBC Driver for Microsoft SQL Server using TDS (Tabular Data Stream) Protocol
Apache License 2.0
183 stars 32 forks source link

Let `Statement.add()` always create a new binding set #230

Closed mp911de closed 2 years ago

mp911de commented 2 years ago

As per r2dbc/r2dbc-spi#229 and r2dbc/r2dbc-spi#252, Statement.add() should always prepare a new binding set to expect a new set of binding parameters. Calling add().execute() would issue a statement without bindings and either the database or the driver should catch this error.