Skip to content

Conversation

yrodiere
Copy link
Member

https://hibernate.atlassian.net//browse/HHH-13266

@gbadner I think we should backport this to 5.3?

yrodiere and others added 18 commits March 14, 2019 08:56
…ion methods between LocalDateTime and Timestamp
…sion methods between OffsetDateTime and Timestamp
…ion methods between ZonedDateTime and Timestamp
So that we can hopefully factorize the upcoming additions.
…storage for timestamps

At least PostgreSQL, Oracle, MySQL and HANA don't support it.
@yrodiere yrodiere changed the title Backport HHH-13266 to branch 5.3 - LocalDateTime values are wrong around 1900 (caused by JDK-8061577) [ON HOLD] Backport HHH-13266 to branch 5.3 - LocalDateTime values are wrong around 1900 (caused by JDK-8061577) Mar 14, 2019
@yrodiere
Copy link
Member Author

Please hold; I had to push a few test fixes for non-H2 databases in #2815. I added the commits to this PR, but let's not backport until #2815 is merged...

@gbadner
Copy link
Contributor

gbadner commented Mar 14, 2019

@yrodiere, thanks for creating this PR!

@yrodiere yrodiere changed the title [ON HOLD] Backport HHH-13266 to branch 5.3 - LocalDateTime values are wrong around 1900 (caused by JDK-8061577) Backport HHH-13266 to branch 5.3 - LocalDateTime values are wrong around 1900 (caused by JDK-8061577) Mar 19, 2019
@yrodiere
Copy link
Member Author

#2815 was merged into branch master and it fixed the failing mariaDB, Oracle and HANA jobs.

We can safely merge this in 5.3 if that's what you want, @gbadner .

…d in some cases due to HHH-13357

            when Time is remapped as Timestamp and default JVM is not GMT
@gbadner
Copy link
Contributor

gbadner commented Apr 19, 2019

I added a commit to skip the failing tests.

@gbadner gbadner merged commit 664d435 into hibernate:5.3 Apr 19, 2019
@yrodiere yrodiere deleted the HHH-13266-5.3 branch June 26, 2019 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants