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

Catch HOUR_OF_DAY error #1160

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Catch HOUR_OF_DAY error #1160

wants to merge 5 commits into from

Commits on Oct 22, 2024

  1. Catch HOUR_OF_DAY error

    If the time stored in the missing daylight saving hour, MySQL JDBC driver starting from version 8.0.23 will raise this error (https://dev.mysql.com/doc/relnotes/connector-j/en/news-8-0-23.html)
    If the date time value stored in the column actually is UTC time, then will return string representation of the value, not the time converted to the local time zone.
    jbaiza committed Oct 22, 2024
    Configuration menu
    Copy the full SHA
    b31ac66 View commit details
    Browse the repository at this point in the history

Commits on Oct 23, 2024

  1. In tests recreate DB connection with actual time zone

    When time zone is changed, JDBC connection still will contain initial time zone. Therefore we need to recreate connection with the newly set TZ
    jbaiza committed Oct 23, 2024
    Configuration menu
    Copy the full SHA
    856ad9c View commit details
    Browse the repository at this point in the history
  2. Add missing skip in DST test

    jbaiza committed Oct 23, 2024
    Configuration menu
    Copy the full SHA
    7e216de View commit details
    Browse the repository at this point in the history
  3. Add pending to test in PostgreSQL case

    With AR default timezone set to :local, PostgreSQL driver will try to convert missing DST time and will fail.
    jbaiza committed Oct 23, 2024
    Configuration menu
    Copy the full SHA
    c623fe2 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    11361a7 View commit details
    Browse the repository at this point in the history