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 2019/01/28 02:15:56 UTC

[GitHub] andyh1203 commented on a change in pull request #4600: [AIRFLOW-3765] Fix for create/update xcom values using the admin UI

andyh1203 commented on a change in pull request #4600: [AIRFLOW-3765] Fix for create/update xcom values using the admin UI
URL: https://github.com/apache/airflow/pull/4600#discussion_r251268601
 
 

 ##########
 File path: airflow/models/__init__.py
 ##########
 @@ -188,6 +188,55 @@ def get_fernet():
     return _fernet
 
 
+class XComSerializer(types.TypeDecorator):
 
 Review comment:
   Hey @seelmann, thanks for the suggestion. That's a good idea. Wanted to check if it also makes sense to move the XCom model / class to this file as well (i.e. airflow/models/xcom.py would contain the XCom class and XComSerializer class).

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services