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

feat: MySQL, MariaDB always store timestamps in UTC #471

Merged
merged 16 commits into from
Mar 11, 2024

Conversation

kagkarlsson
Copy link
Owner

@kagkarlsson kagkarlsson commented Feb 24, 2024

Update:
Breaking change for MySQL and MariaDB users later reverted in #481


  • BREAKING CHANGE: Since MySQL and MariaDB does not have a datatype for storing timestamps with time zones, start defaulting to storing them in UTC to avoid issues with DST
  • BREAKING CHANGE: Change Oracle schema to use a datatype that supports time zones
  • New config-option to override default timestamp persistence behavior for PostgreSQL and Oracle to UTC
  • mysql-jdbc v12.4.2.jre8
  • mariadb-java-client v3.3.2
  • testcontainers v1.19.7

@kagkarlsson kagkarlsson changed the title MySQL, MariaDB always store timestamps in UTC feat: MySQL, MariaDB always store timestamps in UTC Mar 11, 2024
@kagkarlsson kagkarlsson merged commit d422cfb into master Mar 11, 2024
14 checks passed
@kagkarlsson
Copy link
Owner Author

🎉 This issue has been resolved in v14.0.0 (Release Notes)

@kagkarlsson kagkarlsson added the released Issue has been released label Apr 18, 2024
@rydenius
Copy link

rydenius commented May 22, 2024

As it is not mentioned in PR description, was it intentional that there was also an API breaking change for PostgreSQL, with the zero argument constructor of PostgreSqlJdbcCustomization removed?

@kagkarlsson
Copy link
Owner Author

Not intentional no, more of a slip. Sometimes difficult to remember everything that should make it into a release-note or pr-description

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Issue has been released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants