turbaszek removed a comment on pull request #8805: URL: https://github.com/apache/airflow/pull/8805#issuecomment-636327033
> > Could we use the **exit** from the dag to fix this one case (but there are others that wouldn't be caught by that. > > Exactly > > > Given how rare metaprogramming is in Python, I think the metaclass approach gives a more complete solution, it catches all possible ways of assigning deps between a DAG. > > I agree, in our codebase, there's only one operator that will need to be fixed. > > > Yes, this is technically a breaking change, but I think it will affect almost no one. And I think the only "break" would be that using XComArg wouldn't work with a custom operator, unless your metaclass inherits this new metaclass. Is that correct? (i.e. the dag would still run, the user's metaclass would be the one that is used.) > > I think yes > > > So I'm +1 for metaclass, if I've understood it all correctly. > > Happy to bring it back! ---------------------------------------------------------------- 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
