You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/01/21 01:22:18 UTC

[GitHub] [airflow] kaxil commented on pull request #13635: Fix string encoding in DockerOperator when using xcom / json in xcom.py

kaxil commented on pull request #13635:
URL: https://github.com/apache/airflow/pull/13635#issuecomment-764155414


   > I'm not sure about this one. In my opinion the default XCom supports only JSON-serializable types. And bytes are not the type. It's up to an operator to take care of that. If users want to change behaviour of XCom backend they can either use pickle (not recommended for security reasons) or create their own XCom backend:
   > https://airflow.apache.org/docs/apache-airflow/stable/concepts.html?highlight=z#custom-xcom-backend
   
   Agree with @turbaszek , the change in https://github.com/apache/airflow/pull/13536/files is actually what we need


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

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