ashb commented on pull request #10827:
URL: https://github.com/apache/airflow/pull/10827#issuecomment-691927501


   > @casassg @yuqian90 I've added missing method. I'm wondering about making 
`BaseTaskOperation` class that will implement all those methods so we can use 
it in `XComArg`, `BaseOperator` and `TaskGroup` WDYT? @ashb @dimberman @potiuk
   
   Sounds like a prime use of a mixin to me, rather than a base class. But some 
DRYing here now we have it three places definitely sounds good.


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


Reply via email to