potiuk commented on PR #50166: URL: https://github.com/apache/airflow/pull/50166#issuecomment-2849248836
> didn't know that. Did I miss this request? updated. We heavily discussed it last week when I discovered that common.messaging introduces unnecessary coupling and i fixed it to follow the same patterns as we do for other "core-extensions" - i.e. provider's manager. The discussion started here https://github.com/apache/airflow/pull/49938#discussion_r2069008941 with the atempt to add Kafka implementation of the common.messaging provider - it was followed by this discussion https://github.com/apache/airflow/pull/49938#issuecomment-2844687589 and https://github.com/apache/airflow/pull/50057 -> where I moved SQS common.messaging implementation to Amazon provider (where it belongs) and implemented the proper "discover" providers rather than "hard-code" them in common.messaging. That's why it's best comon.messaging and amazon are released together. -- 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]
