[jdbc-v2] setObject(LocalDateTime) applies the JVM timezone and stores a shifted value
还没有人认领这个 Issue。
评估
调研方向
从 jdbc-v2/src/main/java/com/clickhouse/jdbc/PreparedStatementImpl.java 中第 701-710 行附近的 LocalDateTime 处理开始,然后检查 ConnectionImpl.java 中第 107-111 行附近的 defaultCalendar 初始化。使用提供的复现程序测试不同的 JVM 时区和 DateTime64 列。当 setObject(LocalDateTime) 使用目标列或服务器时区保留本地字段,且不发生依赖 JVM 的偏移时,即完成。
由索引模型根据 Issue 内容生成。
描述
Description
JDBC V2 interprets a LocalDateTime passed to PreparedStatement.setObject() using the JVM default timezone.
This happens before the value is sent to ClickHouse. Consequently, the persisted Unix timestamp depends on the timezone of the JVM running the application.
A LocalDateTime does not represent an absolute instant. It contains only local date and time fields. When it is written to a ClickHouse DateTime or DateTime64 column, those fields should be interpreted in the timezone declared by the target column, or in the ClickHouse server timezone when the column has no explicit timezone. The JVM default timezone should not participate in this conversion.
For example:
LocalDateTime: 2019-03-18 10:01:17.123456
JVM timezone: America/Bahia_Banderas (UTC-06:00)
Target column timezone: UTC
The expected interpretation is:
2019-03-18 10:01:17.123456 UTC
JDBC V2 instead interprets it as:
2019-03-18 10:01:17.123456 America/Bahia_Banderas
and sends the corresponding instant:
2019-03-18 16:01:17.123456 UTC
ClickHouse therefore receives and correctly stores an already shifted Unix timestamp.
This was originally found while migrating the Trino ClickHouse connector from the legacy JDBC implementation (0.7.1-patch1) to JDBC V2 (0.10.0), but the reproducer below uses the JDBC driver directly.
Steps to reproduce
- Run ClickHouse with the server timezone set to
UTC. - Set the JVM default timezone to
America/Bahia_Banderas. - Create a
DateTime64(6, 'UTC')column. - Insert a
LocalDateTimeusingPreparedStatement.setObject(). - Query the persisted epoch using
toUnixTimestamp64Micro().
Actual Behaviour
No exception is thrown, but the prepared-statement value is shifted by the JVM timezone offset:
JVM timezone: America/Bahia_Banderas
Server timezone: UTC
SQL literal:
expected=1552903277123456
actual=1552903277123456
setObject(LocalDateTime):
expected=1552903277123456
actual=1552924877123456
difference=21600000000 microseconds
The difference is six hours, matching the JVM timezone offset on the tested date.
Changing only the JVM default timezone to UTC makes the same code produce the expected result.
Expected Behaviour
The local date and time fields of LocalDateTime should be interpreted in the timezone of the target ClickHouse column.
For example:
| Target column | LocalDateTime | Expected stored instant |
|---|---|---|
DateTime64(6, 'UTC') |
2019-03-18 10:01:17.123456 |
2019-03-18 10:01:17.123456 UTC |
DateTime64(6, 'Europe/Moscow') |
2019-03-18 10:01:17.123456 |
2019-03-18 07:01:17.123456 UTC |
In both cases, selecting the value using the column timezone should produce the original local fields:
2019-03-18 10:01:17.123456
For a DateTime or DateTime64 column without an explicit timezone, the server timezone should be used.
The result must not change when the same application is executed with a different JVM default timezone.
If the JDBC driver does not know the target column timezone, it can preserve the LocalDateTime fields as a textual timestamp and let ClickHouse apply the target column or server timezone.
Code Example
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.Statement;
import java.time.LocalDateTime;
import java.time.ZoneOffset;
import java.util.Properties;
import java.util.TimeZone;
public final class ClickHouseLocalDateTimeWriteReproducer
{
private static final LocalDateTime VALUE =
LocalDateTime.of(2019, 3, 18, 10, 1, 17, 123_456_000);
private ClickHouseLocalDateTimeWriteReproducer() {}
public static void main(String[] args)
throws Exception
{
TimeZone originalTimeZone = TimeZone.getDefault();
TimeZone.setDefault(TimeZone.getTimeZone("America/Bahia_Banderas"));
try {
Properties properties = new Properties();
properties.setProperty("user", "default");
properties.setProperty("password", "");
try (Connection connection = DriverManager.getConnection(
"jdbc:clickhouse://localhost:8123/default",
properties);
Statement statement = connection.createStatement()) {
System.out.println("JVM timezone: " + TimeZone.getDefault().getID());
try (ResultSet result = statement.executeQuery("SELECT timezone()")) {
result.next();
System.out.println("Server timezone: " + result.getString(1));
}
statement.execute("DROP TABLE IF EXISTS jdbc_v2_local_datetime_repro");
statement.execute("""
CREATE TABLE jdbc_v2_local_datetime_repro
(
id UInt8,
value DateTime64(6, 'UTC')
)
ENGINE = Memory
""");
// Control row: ClickHouse interprets these local fields using the target column timezone.
statement.execute("""
INSERT INTO jdbc_v2_local_datetime_repro
VALUES (1, '2019-03-18 10:01:17.123456')
""");
// Problematic row: JDBC V2 first interprets LocalDateTime using the JVM default timezone.
try (PreparedStatement preparedStatement = connection.prepareStatement(
"INSERT INTO jdbc_v2_local_datetime_repro VALUES (?, ?)")) {
preparedStatement.setInt(1, 2);
preparedStatement.setObject(2, VALUE);
preparedStatement.executeUpdate();
}
long expected =
VALUE.toEpochSecond(ZoneOffset.UTC) * 1_000_000L
+ VALUE.getNano() / 1_000;
try (ResultSet result = statement.executeQuery("""
SELECT id, toUnixTimestamp64Micro(value)
FROM jdbc_v2_local_datetime_repro
ORDER BY id
""")) {
while (result.next()) {
int id = result.getInt(1);
long actual = result.getLong(2);
System.out.printf(
"id=%d expected=%d actual=%d difference=%d%n",
id,
expected,
actual,
actual - expected);
}
}
}
}
finally {
TimeZone.setDefault(originalTimeZone);
}
}
}
Workaround
Passing the same local date and time as a string preserves the expected fields:
preparedStatement.setString(
parameterIndex,
"2019-03-18 10:01:17.123456");
ClickHouse then interprets the string using the timezone of the target column.
ZonedDateTime is not an equivalent workaround for this use case: ZonedDateTime represents an absolute instant with timezone information, while the original value has SQL TIMESTAMP WITHOUT TIME ZONE semantics.
Suspected Cause
JDBC V2 converts LocalDateTime to a Unix timestamp using the connection's default calendar:
else if (x instanceof LocalDateTime) {
return "fromUnixTimestamp64Nano(" +
DataTypeUtils.toUnixTimestampString(
(LocalDateTime) x,
defaultCalendar.getTimeZone()) +
")";
}
The connection initializes this calendar from the JVM default timezone:
this.defaultCalendar = Calendar.getInstance();
Therefore, the conversion currently behaves approximately as:
localDateTime
.atZone(ZoneId.systemDefault())
.toInstant();
The timezone of the target ClickHouse column is not used.
Configuration
Client Configuration
Properties properties = new Properties();
properties.setProperty("user", "default");
properties.setProperty("password", "");
Connection connection = DriverManager.getConnection(
"jdbc:clickhouse://localhost:8123/default",
properties);
The following properties did not change the result:
use_server_time_zone=false
use_time_zone=UTC
Environment
- Client version:
clickhouse-jdbc 0.10.0 - JDBC implementation: V2 (
com.clickhouse.jdbc.Driver) - Language version: Java
23.0.2 - OS: Linux
- JVM timezone:
America/Bahia_Banderas
ClickHouse Server
- ClickHouse Server version:
24.12.1.1614 - ClickHouse Server timezone:
UTC - ClickHouse Server non-default settings, if any: none relevant
CREATE TABLEstatement:
CREATE TABLE jdbc_v2_local_datetime_repro
(
id UInt8,
value DateTime64(6, 'UTC')
)
ENGINE = Memory;
- 主要语言
- Java
- 星标
- 1.6k
- 派生
- 637
- 平均合并
- 2 天 16 小时
- 30 天内合并 PR
- 30
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
ClickHouse/clickhouse-java 的其他 Issue
-
难度 1/5 1 小时以内 新手友好度 85/100
ClickHouse/clickhouse-java#3111 ·
-
area:data-type bug
难度 2/5 1-3 小时 新手友好度 78/100
ClickHouse/clickhouse-java#3098 · 1 条评论 ·
-
bug client-api-v2 test
难度 2/5 1-3 小时 新手友好度 92/100
ClickHouse/clickhouse-java#3076 ·
-
area:sql-parser bug client-v1
难度 1/5 1-3 小时 新手友好度 92/100
ClickHouse/clickhouse-java#3066 ·
-
area:general bug client-api-v2 jdbc jdbc-v2
难度 2/5 1-3 小时 新手友好度 88/100
ClickHouse/clickhouse-java#3063 ·
查看 ClickHouse/clickhouse-java 的全部 Issue
相似的 Issue
-
area/plugin
难度 2/5 1-3 小时 新手友好度 75/100
kestra-io/plugin-kestra#190 ·
-
难度 2/5 1-3 小时 新手友好度 70/100
google-ai-edge/LiteRT-LM#3739 ·
-
bug
难度 2/5 1-3 小时 新手友好度 75/100
-
难度 2/5 1-3 小时 新手友好度 75/100
integra-team-red/meet-map#249 ·
-
[Studio][Bug] Cancelled create-user dialog keeps the password and admin switch for the next attempt 未关闭
难度 2/5 1-3 小时 新手友好度 75/100
apache/rocketmq-dashboard#5064 ·