Exposed icon indicating copy to clipboard operation
Exposed copied to clipboard

Issue 221 - Map `java.time.Instant` to type stored in UTC

Open tnleeuw opened this issue 3 years ago • 4 comments

Related issue: https://github.com/JetBrains/Exposed/issues/221

Use a timestamp with timezone to store java.time.Instant in UTC where possible; for MySQL use timestamp type which always stores data in UTC.

Works in Progress.

tnleeuw avatar May 04 '21 13:05 tnleeuw

Unit tests failed when I pushed change to remove version-nr update. They passed before that. I will see if I can find out why that is but of course I do not want to merge a change tot the version-nr; perhaps I should rebase on master instead of having branched off of the release 0.31.1 version!

tnleeuw avatar May 12 '21 12:05 tnleeuw

@tnleeuw , hi, can you please check MiscTableTest.testUpdate02 fails on mysql. Failed build

Tapac avatar Aug 08 '21 18:08 Tapac

@tnleeuw , hi, can you please check MiscTableTest.testUpdate02 fails on mysql. Failed build

Hi,

Thanks for having a look at the PR! I will have a look at the failing tests, hopefully tomorrow.

tnleeuw avatar Aug 10 '21 11:08 tnleeuw

This current behavior is very strange! @tnleeuw Please add also TIMESTAMPTZ in PostgreSQLDialect.

hfhbd avatar Nov 02 '21 13:11 hfhbd

Hey @tnleeuw. Thank you for your contribution. This feature has now been implemented here #1787.

joc-a avatar Jul 27 '23 18:07 joc-a

Hey @tnleeuw. Thank you for your contribution. This feature has now been implemented here #1787.

I'm afraid I have not contributed all that much, I never managed to fix unit tests for all supported databases!

tnleeuw avatar Jul 28 '23 09:07 tnleeuw