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 2021/09/15 21:50:05 UTC

[GitHub] [airflow] josh-fell commented on a change in pull request #18278: Updating miscellaneous provider DAGs to use TaskFlow API where applicable

josh-fell commented on a change in pull request #18278:
URL: https://github.com/apache/airflow/pull/18278#discussion_r709599897



##########
File path: airflow/providers/apache/hive/example_dags/example_twitter_dag.py
##########
@@ -137,7 +106,7 @@ def transfertodb():
 
     for channel in to_channels:
 
-        file_name = "to_" + channel + "_" + yesterday.strftime("%Y-%m-%d") + ".csv"
+        file_name = "to_" + channel + "_" + dt + ".csv"

Review comment:
       Yeah we should. I have a habit of going a little deeper on changes that the defined scope sometimes so I was holding myself back. I'll make sure this gets updated.




-- 
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