casassg commented on issue #8052:
URL: https://github.com/apache/airflow/issues/8052#issuecomment-619635122


   Sounds good. Regarding Jinja, maybe a way is to make it `xcom_arg.jinja` 
property instead of making it under __str__. But it's mostly a nitpick, so fine 
by me.
   
   If we keep XComArg as a simple XCom wrapper, that should work. Want to make 
sure we keep XComArg as low complex as possible, and if we need to add some 
extra XCom logic add it to XCom class itself instead.


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to