ClickHouse / clickhouse-java

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

[client-v2] Fixes BinaryStreamReader parsing DateTime64 values. Using a column defined scale now. #1863

Closed chernser closed 1 month ago

chernser commented 1 month ago

Summary

Added more tests for DateTime values and made it be read with column defined scale.

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

Checklist

Delete items not relevant to your PR:

sonarcloud[bot] commented 1 month ago

Quality Gate Failed Quality Gate failed

Failed conditions
40.0% Coverage on New Code (required ≥ 80%)
E Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint