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 2020/11/12 21:54:44 UTC

[GitHub] [airflow] kaxil commented on a change in pull request #12327: Use different deserialization method in XCom init_on_load

kaxil commented on a change in pull request #12327:
URL: https://github.com/apache/airflow/pull/12327#discussion_r522450084



##########
File path: airflow/models/xcom.py
##########
@@ -63,7 +63,7 @@ def init_on_load(self):
         i.e automatically deserialize Xcom value when loading from DB.
         """
         try:
-            self.value = XCom.deserialize_value(self)
+            self.value = XCom.orm_deserialize_value(self)

Review comment:
       why not just change this to `cls.deserialize_value(self)` ? and that way it should use the `deserialize_value` method from Custom backend 




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