mobuchowski commented on PR #27113: URL: https://github.com/apache/airflow/pull/27113#issuecomment-1307350276
@ashb okay. Let's determine first if we need `namespaced` or `scoped` plugin methods. I'm not sure about it - but you and other committers know much better what Airflow needs. Plugins don't have to implement every method - in fact, they can implement whatever they want, one, all or many, so this isn't a problem. Also, I don't see need to physically separate `dag run listener` from `task instance listener`. In my plugin, those would be in the same python module that just provides plumbing to pass the data further, but maybe there is some need for that. If we determine the need we might decide on an implementation, where I would prefer to have logical separation - maybe similar to as we have now - common methods are close together, and can be possibly prefixed with some `namespace` to separate them logically. -- 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]
