amoghrajesh commented on code in PR #46719:
URL: https://github.com/apache/airflow/pull/46719#discussion_r1957178505
##########
airflow/api_fastapi/execution_api/routes/xcoms.py:
##########
@@ -175,7 +160,7 @@ def set_xcom(
task_id: str,
key: str,
value: Annotated[
- str,
+ JsonValue,
Review Comment:
That's true, we can revisit it later again if we need to.
By multi serialisation i meant that we stored a multi serialised value in
the DB. This also lead to a difference from airflow 2 format, which means those
xcoms cant be retrieved.
The current logic does multiple (times) serialisation, but stores singly
serialised value in DB.
--
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]
