Skip to content

Conversation

@amoghrajesh
Copy link
Contributor

Theres a change in behaviour in xcom_pull, for the better and to make it less ambiguous.

Adding to migration docs so folks dont miss it and dont rely on old behaviour.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@amoghrajesh
Copy link
Contributor Author

CC @VladaZakharova

@amoghrajesh amoghrajesh changed the title Adding change in xcom_pull behaviour to migration guide Adding change in xcom_pull behaviour to release notes Apr 22, 2025
Copy link
Member

@kaxil kaxil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

few nits

amoghrajesh and others added 2 commits April 22, 2025 14:59
Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com>
@kaxil kaxil merged commit dda8626 into apache:main Apr 22, 2025
42 checks passed
@MaksYermak
Copy link
Contributor

@amoghrajesh @kaxil what do you think about adding information about tuple serialization and deserialization?
In AF2 when users push this value to xcom [("a", "b"), ("c", "d")] then the result for xcom_pull will be like this [['a', 'b'], ['c', 'd']]. In AF3 this bug was fixed.

kaxil pushed a commit that referenced this pull request Apr 22, 2025
@kaxil kaxil mentioned this pull request Apr 22, 2025
prabhusneha pushed a commit to astronomer/airflow that referenced this pull request Apr 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants