potiuk commented on code in PR #33224: URL: https://github.com/apache/airflow/pull/33224#discussion_r1292201137
########## 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: It's a recursive definition - so every sub-tree and leaf is also a json :) .. BTW. Little known fact (only loosely related). JSON is a 100% compatible subset of YAML. Every JSON is also a valid YAML file, and you can use YAML parser to load JSON file. I know it's surprising but it's true. -- 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]
