dstandish commented on code in PR #59115:
URL: https://github.com/apache/airflow/pull/59115#discussion_r2713368474
##########
airflow-core/src/airflow/api/common/mark_tasks.py:
##########
@@ -70,6 +70,9 @@ def set_state(
:param commit: Commit tasks to be altered to the database
:param session: database session
:return: list of tasks that have been created and updated
+
+ TODO: "past" and "future" params currently depend on logical date, which
is not always populated.
+ we might want to just deprecate these options. Or alter them to do
*something* in that case.
Review Comment:
yes, i think we need to add partition date as a field on dag run -- i'm
avoiding adding that in this pr though just cus it's already huge
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
