paultmathew commented on PR #67226: URL: https://github.com/apache/airflow/pull/67226#issuecomment-4512090401
Thanks for the review @jscheffl. Pushed a fix with an operator-local _push_xcom_with_fan_out helper used by both sync (post_complete_action) and async (trigger_reentry) failure-path pushes — so all four code paths now honour multiple_outputs consistently. Added matching failure-path tests for both modes. I considered promoting _push_xcom_if_needed to a public task-SDK helper as you mentioned. Manual XCOM_RETURN_KEY pushes from operator code are essentially KPO-specific today (grep across all providers only turns up one other place), so I kept the helper local to keep this PR focused — happy to file a follow-up issue for the wider task-SDK change if you'd prefer. -- 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]
