turbaszek commented on pull request #12327: URL: https://github.com/apache/airflow/pull/12327#issuecomment-726824591
Thinking about it I lean towards hardcoding the `init_on_load` with `BaseXCom.deserialize_value` - in this way we will make sure that users do not break internal stuff. As an addition to that I would add note in docs that is is required to call `BaseXCom` methods in custom `serialize_value` and `deserialize_value` to make sure that the value we store in **xcom tabel** is json serialisable (somehow discouraging users to use pickle + db as storage or anything like that). What do you think @ashb @kaxil ? ---------------------------------------------------------------- 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: [email protected]
