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.

Sign up for GitHub

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

Jump to bottom

DefaultConverterProvider should be able to convert both ISO timestamp formats to Timestamp and LocalDateTime #12131

Closed
lukaseder opened this issue Jul 8, 2021 · 3 comments
Closed

DefaultConverterProvider should be able to convert both ISO timestamp formats to Timestamp and LocalDateTime #12131

lukaseder opened this issue Jul 8, 2021 · 3 comments
Labels
C: Functionality E: All Editions P: Medium R: Fixed T: Defect
Milestone
Version 3.16.0

Comments

@lukaseder
Copy link
Member

Both of these are valid ISO 8601 timestamps

  • 2000-03-15T10:15:00 (default syntax, as used in XML, java.time)
  • 2000-03-15 10:15:00 (optional syntax, as used in SQL, java.sql)

Our DefaultConverterProvider should be able to parse both to both java.time.LocalDateTime and java.sql.Timestamp
@lukaseder lukaseder added this to the Version 3.16.0 milestone Jul 8, 2021
@lukaseder
Copy link
Member Author

This should fix #12130

@lukaseder
Copy link
Member Author

We already support parsing the SQL format to java.time

@lukaseder
Copy link
Member Author

Fixed in jOOQ 3.16.0 and 3.15.1 (#12133)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C: Functionality E: All Editions P: Medium R: Fixed T: Defect
Projects
None yet
Milestone
Version 3.16.0
Development

No branches or pull requests
1 participant
@lukaseder