vemikhaylov commented on PR #29608:
URL: https://github.com/apache/airflow/pull/29608#issuecomment-1439054387

   Regarding the JSON direction potentially a better way would be to make it 
more explicitly structured like:
   
   ```
   {"get_python_echo_message": {"key": "return_value", "value": "test xcom 
arg"}}
   ```
   or just
   ```
   [{"task_id": "get_python_echo_message", "key": "return_value", "value": 
"test xcom arg"}, ...]
   ```
   
   `key` may be optional with `"return_value"` as the default.
   
   ```
   [{"task_id": "get_python_echo_message", "value": "test xcom arg"}, ...]
   ```
   
   It may look a bit more wordy and cumbersome, but at the same time clearer 
and more extensible in the future. Before adjusting the implementation, I would 
like to get more eyes on these to see if there are any concerns or if there are 
good alternatives to the JSON way.


-- 
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-unsubscr...@airflow.apache.org

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

Reply via email to