fredthomsen commented on PR #62079: URL: https://github.com/apache/airflow/pull/62079#issuecomment-3939691611
Reviewed the concerns raised on #55257, which stalled because it bypassed the existing `operator_extra_links` serialization pipeline — it used a plain `dict` pushed in `finalize()` and patched the webserver API with XCom wildcard scans, raising concerns about key collisions and split data sources. This PR sidesteps those issues by creating `TaskFlowExtraLink` objects at parse time via `@task(extra_links=["Name"])`, plugging directly into the existing `operator_extra_links` → serialization → `XComOperatorLink` pipeline. Zero webserver changes needed. URLs are pushed immediately via `SUPERVISOR_COMMS` so `@task.virtualenv` works transparently. The tradeoff is that link names must be pre-declared in the decorator rather than created dynamically at runtime. cc @ashb @pierrejeambrun -- 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]
