lanxinxs commented on issue #51521: URL: https://github.com/apache/airflow/issues/51521#issuecomment-2960988909
> Hi [@lanxinxs](https://github.com/lanxinxs) [@jroachgolf84](https://github.com/jroachgolf84) [@sjyangkevin](https://github.com/sjyangkevin) — > > I’d like to **take ownership of this fix**. Plan: > > * ✅ First, reproduce the error on 3.0.1 to confirm the validator path. > * 🔧 Patch: if `key is None`, translate to the literal `"return_value"` (restoring Airflow 2 behaviour). > * 🧪 Add a unit test covering the legacy case. > * 📝 Tweak the docstring so future readers know `None → "return_value"`. > > Two-click reply: > > 1. 👍 = “sounds good, go for it” > 2. ➡️ Anything else you’d like double-checked (e.g., back-port label for 3.0.3)? > > Happy to adjust if you see a better approach—just let me know. > > Thanks for all your work on Airflow! > > * Sebastian (UTC-7) my expected behaviour is: if `key is None`, translate to 'get all xcoms no matter what their keys are', that is, if run_id is given, then pull all xcoms in this dag run; if run_id and task_ids are given, then pull all xcoms within these tasks in this dag run. -- 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]
