Skip to content

Add proper dependencies for sqlalchemy - dependent providers #59895

Closed
14 of 14 issues completed
Closed

Description

@potiuk

Some of our providers have direct imports from sqlalchemy:

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.

The way how it should be done:

  • for those providers that "need" sqlalchemy badly - sqlalchemy (same specification as in airflow-core now) should be added as "required" dependency

  • for those providers where sqlalchemy is just for some feature, we should make sure that they handle lack of sqlalchemy installed - handing import error, and raising "Optional Provider Feature" exception when a method that needs sqlalchemy is callsed and sqlalchemy is not installed. Sqlalchemy should be added as "extra" sqlalchemy dependency.

Activity

  1. converted this from a draft issue on Dec 29, 2025
  2. 2 remaining items

  3. mohit7705 commented on Dec 29, 2025

    @mohit7705
    Contributor

    Hi @potiuk 👋

    I’d like to work on Add proper dependencies for sqlalchemy – postgres (#59905) if it’s still available.
    I’m new to Airflow contributions but familiar with Python dependency management and optional imports.

    Please let me know if I can proceed. Thanks!

  4. moved this from Ready to No status in CI / DEV ENV planned workon Dec 29, 2025
  5. moved this from Ready to In progress in CI / DEV ENV planned workon Dec 29, 2025
  6. self-assigned this
    on Dec 29, 2025
  7. potiuk commented on Dec 29, 2025

    @potiuk
    MemberAuthor

    Hi @potiuk 👋

    I’d like to work on Add proper dependencies for sqlalchemy – postgres (#59905) if it’s still available. I’m new to Airflow contributions but familiar with Python dependency management and optional imports.

    Please let me know if I can proceed. Thanks!

    Please comment in specific issue and I will assign you. I cannot assign you if you do not comment on the issue to be assigned to.

  8. sunank200 commented on Dec 30, 2025

    @sunank200
    Collaborator

    @potiuk I can work on Amazon, Impala, Open Lineage and Edge3.

  9. sunank200 commented on Dec 30, 2025

    @sunank200
    Collaborator

    Seems like they are already assigned. If those are not picked up, I can take it up.

  10. potiuk commented on Dec 30, 2025

    @potiuk
    MemberAuthor

    I think all of the sub-issues are assigned already - unless of course someone wants to give it to @sunank200 :).

    Image
  11. potiuk commented on Jan 4, 2026

    @potiuk
    MemberAuthor

    Seems like they are already assigned. If those are not picked up, I can take it up.

    @sunank200 -> databricks one is u for grabs now.

  12. eladkal commented on Mar 29, 2026

    @eladkal
    Contributor

    I think all tasks in this issue are resolved?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions