Skip to content
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

SNOW-1723202: Support Java 8 java.time types #169

Open
DenWav opened this issue Oct 5, 2024 · 0 comments
Open

SNOW-1723202: Support Java 8 java.time types #169

DenWav opened this issue Oct 5, 2024 · 0 comments

Comments

@DenWav
Copy link

DenWav commented Oct 5, 2024

Java 8 time types are more modern and should be preferred over the old types Date, Time, etc.

  • java.time.LocalDate
  • java.time.LocalTime
  • java.time.LocalDateTime
  • java.time.OffsetDateTime
  • java.time.ZonedDateTime
  • java.time.Instant

The Snowflake documentation specifies:

Although Snowflake can store time values with nanosecond precision, the java.sql.time library maintains only millisecond precision. Conversion between Snowflake and Java data types can reduce effective precision to milliseconds.

This limitation is not present in the modern java.time types, and time zone information is explicit, rather than implicit and prone to error.

Note that proper implementation of this would not involve converting from the old Java types to the new ones, since that would not fix the issue described here.

@github-actions github-actions bot changed the title Support Java 8 java.time types SNOW-1723202: Support Java 8 java.time types Oct 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant