ataulmujeeb-cyber commented on PR #65240:
URL: https://github.com/apache/airflow/pull/65240#issuecomment-4246947109
Hi @jroachgolf84,
Thanks for the feedback and for pointing me to the original implementation
in PR #54383
I see the original approach was integrated into _get_task_instances with an
include_dependent_dags parameter, which recursively discovered
ExternalTaskMarker tasks and gathered their external TIs as part of the query
itself rather than as a post-clearing step like my approach.
Happy to align with your approach if you already have a PR in progress.
A couple of questions:
1. Would you like me to close this PR in favor of yours, or would you prefer
I rework this to follow the original pattern?
2. Could you share a link to your PR so I can review and potentially help?
--
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]
