Mr-Neutr0n commented on PR #61852: URL: https://github.com/apache/airflow/pull/61852#issuecomment-3896692170
Thanks for catching that @ecodina! You're right — I was only handling the `execution_delta` case. I've pushed an update that also handles `execution_date_fn`. When it's set, the link resolution now calls it with the current logical date to determine the target date for the external DAG run lookup. It also handles the case where `execution_date_fn` returns a list (uses the first element) vs a single datetime. Since `execution_date_fn` and `execution_delta` are mutually exclusive, I added it as an `elif` branch. The whole thing is still wrapped in the existing try/except, so if `execution_date_fn` raises (e.g., because it expects full context kwargs we don't have in the link), it gracefully falls back to using the current run_id. -- 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]
