Skip to content

Update SQLAlchemy documentation links from 1.4 (en/14) to 2.0 (en/20) #69164

Closed

Description

@shahar1

Human Summary

While reviewing #69150 (which fixes #25461), I noticed that a number of our
SQLAlchemy documentation links still point at the SQLAlchemy 1.4 docs (en/14), even though
airflow-core now requires SQLAlchemy 2.x .

I don't usually open issues for documentation, but this one is better handled by
someone with SQLAlchemy domain knowledge.

Clarification: This is not intended to be a simple 1.4 -> 2.0 replacement - the entire documentation should also be reviewed to ensure it reflects any critical changes between the 1.4 and 2.0 versions


AI Summary

What

Several SQLAlchemy documentation links across the codebase still point at the
SQLAlchemy 1.4 docs (https://docs.sqlalchemy.org/en/14/...), even though
airflow-core now requires SQLAlchemy 2.x:

  • airflow-core/pyproject.toml pins sqlalchemy[asyncio]>=2.0.48
  • uv.lock resolves to sqlalchemy 2.0.51

The SQLAlchemy 1.4 constraint that originally drove some of these references is
gone, so the user- and developer-facing doc links should point at the en/20/
(2.0) docs that match the version Airflow actually runs.

Acceptance criteria

  • All nine links above use en/20 and still resolve to a live page + anchor.
  • set-up-database.rst:201 is left untouched.
  • breeze build-docs passes (no broken external-link / spellcheck failures).

Notes

Doc/comment-only — no behavior change, no newsfragment needed (the
providers/sqlite comments are a provider change, so no newsfragment there
either).


Drafted-by: Claude Code (Opus 4.8); AI Summary section compiled at the maintainer's request

Activity

  1. Dev-iL commented on Jun 30, 2026

    @Dev-iL
    Collaborator

    airflow-core/docs/howto/set-up-database.rst:201 —
    https://docs.sqlalchemy.org/en/14/changelog/changelog_14.html#change-3687655465c25a39b968b4f5f6e9170b

    This points at a specific SQLAlchemy 1.4 changelog entry; the anchor only
    exists under en/14. Leave it as-is.

    This is false. A en/20 path exists below:

    https://docs.sqlalchemy.org/en/20/changelog/changelog_14.html#change-3687655465c25a39b968b4f5f6e9170b
    
  2. kalyanram07 commented on Jun 30, 2026

    @kalyanram07

    Hi @shahar1! I would love to fix these documentation links for my first contribution. Please assign this issue to me!

  3. shahar1 commented on Jun 30, 2026

    @shahar1
    ContributorAuthor

    airflow-core/docs/howto/set-up-database.rst:201 —
    https://docs.sqlalchemy.org/en/14/changelog/changelog_14.html#change-3687655465c25a39b968b4f5f6e9170b
    This points at a specific SQLAlchemy 1.4 changelog entry; the anchor only
    exists under en/14. Leave it as-is.

    This is false. A en/20 path exists below:

    https://docs.sqlalchemy.org/en/20/changelog/changelog_14.html#change-3687655465c25a39b968b4f5f6e9170b
    

    Got sloppy here, removed it from the AI summary.

    Hi @shahar1! I would love to fix these documentation links for my first contribution. Please assign this issue to me!

    Hey! Feel free to raise a PR to fix it - currently we don't assign people to issues by policy.

  4. AshleyAHuang commented on Jul 1, 2026

    @AshleyAHuang
    Contributor

    There are now three open PRs addressing this issue:

    Flagging so a maintainer can pick whichever is preferred and close the
    others per the "better PR wins" convention — happy to close mine in favor
    of either of the other two if one is extended to cover the missing file.


    Drafted-by: Claude Code (Sonnet 5); reviewed by @AshleyAHuang before posting

  5. added this to the Airflow 3.3.1 milestone on Jul 1, 2026
  6. reopened this on Jul 1, 2026
  7. pallaviB463 commented on Jul 4, 2026

    @pallaviB463

    Hi! I'd like to work on this issue. Could I be assigned?

  8. shahar1 commented on Jul 4, 2026

    @shahar1
    ContributorAuthor

    Hi! I'd like to work on this issue. Could I be assigned?

    Hey! Feel free to raise a PR to fix it - currently we don't assign people to GitHub issues by policy.

  9. added a commit that references this issue on Jul 8, 2026
    a5621ef
  10. vishal-1401 commented on Jul 25, 2026

    @vishal-1401

    Hi, I'd like to work on this issue. Could you please assign it to me?

  11. shahar1 commented on Jul 25, 2026

    @shahar1
    ContributorAuthor

    Hi, I'd like to work on this issue. Could you please assign it to me?

    Feel free to raise a PR, ,there are no official assignments by policy

  12. vatsrahul1001 commented on Jul 28, 2026

    @vatsrahul1001
    Contributor

    Both #69168 and #69168 need work to be done. Moving this to 3.3.2

  13. LeonxLJX commented on Aug 31, 2026

    @LeonxLJX

    Hi! I'd like to update the SQLAlchemy documentation links from 1.4 to 2.0. Thanks!

  14. LeonxLJX commented on Sep 1, 2026

    @LeonxLJX

    Hi! I'd like to work on this. I'll trace the failing path in Airflow, write a regression test, implement the fix and ensure the related unit tests pass. Please let me know if I can proceed / be assigned.

  15. sreecharan-desu commented on Sep 2, 2026

    @sreecharan-desu
    Contributor

    Data point from a fresh main checkout: grep for docs.sqlalchemy.org/en/14 is clean now. Might be worth double-checking whether #69168 / open drafts still diff anything before more link-update PRs pile up.

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

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions