uranusjr commented on PR #69426: URL: https://github.com/apache/airflow/pull/69426#issuecomment-5113677218
I don’t like this. Looking at how classic operators do this (with BaseOperatorLink), the link generation logic is separated from the operator itself, not entangled into the task execution logic. I would suggest doing this more similar to classic operators instead—maybe we can add a “task kind” field that can replace the `operators` field in BaseOperatorLink, so this can be more granular than per-operator-type (which does not suit taskflow well). There are many possibilities, but I don’t think this is a good design. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
