-
Notifications
You must be signed in to change notification settings - Fork 537
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WIP - Lower the datetime limit to what CH also accepts - 1900-01-01 #1386
base: main
Are you sure you want to change the base?
WIP - Lower the datetime limit to what CH also accepts - 1900-01-01 #1386
Conversation
7863ce7
to
17f43db
Compare
Thanks @abdelhakimbendjabeur for the pull request. I linked an issue whifh contains more information that you may want to check. Ideally, we probably should drop range check in Java client and only do that JDBC driver. |
https://clickhouse.com/docs/en/sql-reference/data-types/datetime64 Note: The precision of the maximum value is 8. If the maximum precision of 9 digits (nanoseconds) is used, the maximum supported value is 2262-04-11 23:47:16 in UTC. |
Hi @den-crane , in the docs it says the upper limit is |
@abdelhakimbendjabeur Note: The precision of the maximum value is 8. |
Summary
Removing the layer that performs whether a datetime is within a predefined range works fine.
Checklist
Delete items not relevant to your PR: