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/02/21 07:46:45 UTC

[GitHub] [airflow] potiuk commented on issue #7450: [AIRFLOW-6829][WIP] Introduce BaseOperatorMetaClass - auto-apply apply_default

potiuk commented on issue #7450: [AIRFLOW-6829][WIP] Introduce BaseOperatorMetaClass - auto-apply apply_default
URL: https://github.com/apache/airflow/pull/7450#issuecomment-589538133
 
 
   I think we will eventually have to slightly modify the source code of the operators when preparing backport packages indeed and In this case I agree monkey patching is worse idea :). 
   
   The idea is to have class deriving from the BaseOperator and implement MetaClass and have all the backported operators to derive from this class instead. We can do it by modifying the source code (slightly hacky) or by modifying the Concrete Syntax Tree via https://pybowler.io/

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


With regards,
Apache Git Services