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 2022/02/01 04:36:39 UTC

[GitHub] [airflow] uranusjr commented on pull request #20508: Fix dates.days_ago to respect localized timezone configuration.

uranusjr commented on pull request #20508:
URL: https://github.com/apache/airflow/pull/20508#issuecomment-1026471377


   Cross-posting my response on the mailing list in a shorter form, the problem of `days_ago` is not (at least not entirely) that it’s not frequently used, but _the fact of it being used_ provides little advantage and ironically is a minor foot gun that almost never does what you intend to. Yes, even your usage of `start_date=days_ago(7)` is strongly discouraged because the expression is too ambiguous and leads to confusing behaviour down the road.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org