potiuk opened a new issue, #59895: URL: https://github.com/apache/airflow/issues/59895
Some of our providers have direct imports from sqlalchemy: * [ ] amazon * [ ] drill * [ ] mpala * [ ] common.sql * [ ] databricks * [ ] edge3 * [ ] exasol * [ ] fab * [ ] openlineage * [ ] postgres * [ ] presto * [ ] teradata * [ ] vertica * [ ] hive (soon) The sqlalchemy dependency there is usualy for very small part of the functionality of those providers, so it should be "Optional Provider Feature" rather than being effectively "mandatory" dependency. This is not a problem today - but when we have task isolation in place, task-sdk will have no sqlalchemy as dependency, so those providers should have optional sqlalchemy extra + handling importing the hooks when sqlalchemy is not installed (i.e. either raising OptionalProviderFeatureException when used, and handling fallback to import errors when sqlalchemy import fails. -- 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]
