airflow icon indicating copy to clipboard operation
airflow copied to clipboard

Fix double serialization in XCom PATCH API endpoint

Open Arunodoy18 opened this issue 3 weeks ago • 2 comments

The update_xcom_entry endpoint was calling XComModel.serialize_value twice, causing values to be excessively serialized. This made the updated XCom values unusable when retrieved by tasks.

Changes:

  • Removed duplicate XComModel.serialize_value call in update_xcom_entry
  • Fixed existing test to remove double serialization in assertion
  • Added comprehensive test to verify patched values are correctly serialized and can be retrieved/deserialized properly

Fixes: https://github.com/apache/airflow/issues/59032


^ 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.

Arunodoy18 avatar Dec 06 '25 09:12 Arunodoy18

I have solved the issue , please assure reviewing the issue which is solved, If any thing is needed which is to be changed , please tell. Thank you .

Arunodoy18 avatar Dec 06 '25 09:12 Arunodoy18

Needs rebase + conflict solving.

pierrejeambrun avatar Dec 09 '25 16:12 pierrejeambrun