[client-v2] Tests and Fixes in scope of SerDe

ClickHouse / clickhouse-java

ClickHouse Java Clients & JDBC Driver

https://clickhouse.com

Apache License 2.0

1.44k stars 530 forks source link

[client-v2] Tests and Fixes in scope of SerDe #1830

Closed chernser closed 4 days ago

chernser commented 1 week ago

Summary

Added more data type tests
Fixed read error from response
Added possibility to register class against different tables.

Checklist

Delete items not relevant to your PR:

[x] Unit and integration tests covering the common scenarios were added
[ ] A human-readable description of the changes was provided to include in CHANGELOG
[ ] For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

sonarcloud[bot] commented 5 days ago

Quality Gate failed

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

See analysis details on SonarCloud

chernser commented 5 days ago

I've found that reading AggregationFunction seems has some bug in reading code what causes reading far after real stream ended. I will work in this issue later. Currently it is only for writing.