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

Investigate support of Java8 java.time* classes #539

Open
tomuben opened this issue Mar 17, 2022 · 2 comments
Open

Investigate support of Java8 java.time* classes #539

tomuben opened this issue Mar 17, 2022 · 2 comments
Labels
feature Product feature

Comments

@tomuben
Copy link
Collaborator

tomuben commented Mar 17, 2022

Background

The implementation of the java.sql.Timestamp causes some confusion today, because this class might contain a timezone (extends java.util.Date). The timezone is implicitly set in the udf client and depends of the timezone set on the DB node.

The Java 8 java.time.Instant and java.time.LocalDateTime come without a timezone and would match the Exasol DB TIMESTAMP type. java.time.ZonedDateTime would be the best match for Exasol DATE.

Acceptance Criteria

Investigate and implement support for

  • java.time.Instant
  • java.time.ZoneDateTime
  • java.time.LocalDateTime
@tomuben tomuben added the feature Product feature label Mar 17, 2022
@tomuben
Copy link
Collaborator Author

tomuben commented Mar 17, 2022

timezone

@tomuben
Copy link
Collaborator Author

tomuben commented Mar 21, 2022

  1. Also need to investigate if we should change behavior of old classes. This would mean a breaking change for customers.
  2. Need to involve core-db here. We would need to pass the TZ info to the UDF client from the Database.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Product feature
Projects
None yet
Development

No branches or pull requests

1 participant