You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I noticed this thing while implementing some changes in my project, regarding #378. It has not been a problem before, since the only date column in the messages table is auto-generated by the database engine. If we eventually decide to add support for delayed messages, this will erase the time zones of future execution times and may lead to problems when interpreting those times back in Go code.
The standard practice is to use TIMESTAMP WITH TIME ZONE (or the MySQL equivalent) and rely on timestamp-enabled times provided by clients.
I am just pointing this out for now. Even though it is a small change, it is not something that can be switched right away, as I am not 100% about the effect it will have on existing schemas. Most likely, plain timestamps will be interpreted as UTC, but this needs to be tested on both Postgres and MySQL.
The text was updated successfully, but these errors were encountered:
I noticed this thing while implementing some changes in my project, regarding #378. It has not been a problem before, since the only date column in the messages table is auto-generated by the database engine. If we eventually decide to add support for delayed messages, this will erase the time zones of future execution times and may lead to problems when interpreting those times back in Go code.
The standard practice is to use TIMESTAMP WITH TIME ZONE (or the MySQL equivalent) and rely on timestamp-enabled times provided by clients.
I am just pointing this out for now. Even though it is a small change, it is not something that can be switched right away, as I am not 100% about the effect it will have on existing schemas. Most likely, plain timestamps will be interpreted as UTC, but this needs to be tested on both Postgres and MySQL.
The text was updated successfully, but these errors were encountered: