ClickHouse / clickhouse-java

ClickHouse Java Clients & JDBC Driver
https://clickhouse.com
Apache License 2.0
1.45k stars 535 forks source link

[client-v2] fixed number conversion in reader #1856

Closed chernser closed 2 weeks ago

chernser commented 1 month ago

Summary

This PR implement conversion between different number types. Today client API (readers) allow to get column value only in the type it is defined in a DB. It is very inconvenient because requires to follow schema very strictly. So this PR relaxes this rule and lets to:

Closes: https://github.com/ClickHouse/clickhouse-java/issues/1852

Checklist

Delete items not relevant to your PR:

sonarcloud[bot] commented 2 weeks ago

Quality Gate Failed Quality Gate failed

Failed conditions
49.7% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

chernser commented 2 weeks ago

@mzitnik I may not tell exact numbers - will try running benchmark tests.