Skip to content

Comments

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

Closed
tnleeuw wants to merge 6 commits intoJetBrains:mainfrom
tnleeuw:ISSUE-221-TimestampWithTimezone
Closed

Issue 221 - Map java.time.Instant to type stored in UTC#1223
tnleeuw wants to merge 6 commits intoJetBrains:mainfrom
tnleeuw:ISSUE-221-TimestampWithTimezone

Conversation

@tnleeuw
Copy link

@tnleeuw tnleeuw commented May 4, 2021

Related issue: #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 tnleeuw marked this pull request as ready for review May 6, 2021 10:48
@tnleeuw
Copy link
Author

tnleeuw commented May 12, 2021

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 tnleeuw force-pushed the ISSUE-221-TimestampWithTimezone branch from 672b5e2 to d8a28df Compare May 12, 2021 12:31
Tapac added 2 commits August 8, 2021 20:17
# Conflicts:
#	exposed-java-time/src/main/kotlin/org/jetbrains/exposed/sql/java-time/JavaDateColumnType.kt
#	gradle.properties
@Tapac
Copy link
Contributor

Tapac commented Aug 8, 2021

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

@tnleeuw
Copy link
Author

tnleeuw commented Aug 10, 2021

@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.

@hfhbd
Copy link
Contributor

hfhbd commented Nov 2, 2021

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

@joc-a
Copy link
Contributor

joc-a commented Jul 27, 2023

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

@joc-a joc-a closed this Jul 27, 2023
@tnleeuw
Copy link
Author

tnleeuw commented Jul 28, 2023

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 tnleeuw deleted the ISSUE-221-TimestampWithTimezone branch July 28, 2023 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants