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

Why do you use java.time.Instant? #18

Closed
mjza opened this issue Jul 8, 2020 · 3 comments
Closed

Why do you use java.time.Instant? #18

mjza opened this issue Jul 8, 2020 · 3 comments
Assignees
Labels
invalid This doesn't seem right

Comments

@mjza
Copy link
Contributor

mjza commented Jul 8, 2020

I would suggest to use java.util.Date instead of java.time.Instant in DateAudit.java because some libraries like Olingo JPA does not support java.time.Instant

@isopropylcyanide
Copy link
Owner

I think this article should sum up why java.util.date is deprecated.

As for the compatibility with Olingo API, I'm not sure why it would not be compatible? A java.time.Instant can be converted to a java.util.date

@mjza
Copy link
Contributor Author

mjza commented Jul 9, 2020

I tried to use a converter even but it was not run automatically as claimed in this tutorial:

https://www.adam-bien.com/roller/abien/entry/new_java_8_date_and

At the moment I have no clew why the JPA does not support java.time.Instant, hence I have changed them manually to java.util.date.

@mjza
Copy link
Contributor Author

mjza commented Jul 14, 2020

@mjza mjza closed this as completed Jul 29, 2020
@isopropylcyanide isopropylcyanide added the invalid This doesn't seem right label Oct 30, 2021
@isopropylcyanide isopropylcyanide self-assigned this Oct 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

2 participants