rernas35
rernas35
Strings multiply solution is added.
This reminds that probably we need to promote FakeTransaction from clickhouse-jdbc to clickhouse-client, so that it can be reused in R2DBC as well. This is something we need to refactor...
We need to declare a constant to reuse in both JDBC and R2DBC driver. This can be updated later. https://github.com/rernas35/clickhouse-jdbc/blob/41e1afc89c4c5aa6c1e6874148545086deca9d06/clickhouse-r2dbc/src/main/java/com/clickhouse/r2dbc/connection/ClickHouseConnectionMetadata.java#L28
Fetch size should impact buffer size and maybe maximum queued buffers etc. I can fix this together with JDBC driver after merging the initial PR. https://github.com/rernas35/clickhouse-jdbc/blob/41e1afc89c4c5aa6c1e6874148545086deca9d06/clickhouse-r2dbc/src/main/java/com/clickhouse/r2dbc/ClickHouseStatement.java#L129
navigator.UserAgent won't be valid in future. Do you have plan to support Client Hints ? https://web.dev/migrate-to-ua-ch/