You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "Lee-W (via GitHub)" <gi...@apache.org> on 2023/08/12 08:12:19 UTC

[GitHub] [airflow] Lee-W commented on a diff in pull request #33224: doc: KPO - xcom - only valid json content

Lee-W commented on code in PR #33224:
URL: https://github.com/apache/airflow/pull/33224#discussion_r1292150097


##########
docs/apache-airflow-providers-cncf-kubernetes/operators.rst:
##########
@@ -152,6 +152,10 @@ XCom values differently than other operators. In order to pass a XCom value
 from your Pod you must specify the ``do_xcom_push`` as ``True``. This will create a sidecar container that runs
 alongside the Pod. The Pod must write the XCom value into this location at the ``/airflow/xcom/return.json`` path.
 
+.. note::
+  An invalid json content will fail, example ``echo 'hello' > /airflow/xcom/return.json`` fail and  ``echo '\"hello\"' > /airflow/xcom/return.json`` work

Review Comment:
   May I know why is '"hello"' an valid json content?



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org