iRAFEEK commented on PR #70343: URL: https://github.com/apache/airflow/pull/70343#issuecomment-5208995532
@aaron-y-chen good catch, you're right. `finalize()` was pushing to XCom unconditionally after `get_link()`, so a `static_url` link was still persisted every run. Fixed in 8349e44: skip the push when the link is static. Safe because the render path (`SerializedBaseOperator.get_extra_links`) calls `get_link()` directly and never reads that XCom back. Dynamic links are unchanged; added a test covering both. -- 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]
