ashb commented on a change in pull request #15389:
URL: https://github.com/apache/airflow/pull/15389#discussion_r639695257
##########
File path: airflow/models/dag.py
##########
@@ -1223,7 +1244,7 @@ def clear(
include_subdags=include_subdags,
include_parentdag=False,
dag_run_state=dag_run_state,
- get_tis=True,
+ get_ti_keys=True,
Review comment:
So I'm a little bit confused as to why we had to change this method in
this PR at all.
The only place this is called with `get_ti_keys` is in this recursive use
here.
So I guess what I'm asking is what in this deferrable PR caused a change in
behaviour to the clear function necessitating this change?
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
