clickhouse-java icon indicating copy to clipboard operation
clickhouse-java copied to clipboard

[jdbc-v2] Potentially wrong encoding for java.sql.Date

Open Onajk opened this issue 7 months ago • 1 comments

Description

I have faced some type conversion problems during serializing java.sql.Date in jdbc-v2. jdbc V2 converts java.sql.Date to java.time.Instant and serialise it using CH fromUnixTimestamp64Nano function which returns DateTime64 type but I was expecting Date.

Date is mapped into Instant but in if-else you could see branch for Date, isn't there a mistake? https://github.com/ClickHouse/clickhouse-java/blob/main/jdbc-v2/src/main/java/com/clickhouse/jdbc/PreparedStatementImpl.java#L427

https://github.com/ClickHouse/clickhouse-java/blob/main/jdbc-v2/src/main/java/com/clickhouse/jdbc/PreparedStatementImpl.java#L746

Onajk avatar May 22 '25 09:05 Onajk

Good day, @Onajk ! I think so, I will look into the problem.

chernser avatar Jul 12 '25 06:07 chernser