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/03/16 10:03:31 UTC

[GitHub] [airflow] suiting-young commented on a change in pull request #14827: Fixed default XCom deserialization.

suiting-young commented on a change in pull request #14827:
URL: https://github.com/apache/airflow/pull/14827#discussion_r595021082



##########
File path: airflow/models/xcom.py
##########
@@ -62,13 +62,7 @@ def init_on_load(self):
         Called by the ORM after the instance has been loaded from the DB or otherwise reconstituted
         i.e automatically deserialize Xcom value when loading from DB.
         """
-        try:
-            self.value = self.orm_deserialize_value()
-        except (UnicodeEncodeError, ValueError):

Review comment:
       Those exceptions came from Python2 `json.loads`, have been replaced by `JSONDecodeError` (_as below_) in Python3.




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