You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2018/10/02 08:54:21 UTC
[GitHub] ashb commented on issue #3985: [AIRFLOW-3138] Use current data type
for migrations
ashb commented on issue #3985: [AIRFLOW-3138] Use current data type for migrations
URL: https://github.com/apache/incubator-airflow/pull/3985#issuecomment-426199116
@bolkedebruin This looks okay to me - do you have time for a quick check too.
I think this is okay as our UtcDateTime type is the same column type under the hood. But thinking a bit more about it does doing this set the timezones incorrectly on the row. Hmmmmm.
(Turns out there's no way to remove a review approval in github)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services