Skip to content

Update SQLAlchemy pooling link from 1.4 to 2.0#69857

Closed
br413 wants to merge 2 commits into
apache:mainfrom
br413:docs/sqlalchemy-2.0-pooling-link
Closed

Update SQLAlchemy pooling link from 1.4 to 2.0#69857
br413 wants to merge 2 commits into
apache:mainfrom
br413:docs/sqlalchemy-2.0-pooling-link

Conversation

@br413

@br413 br413 commented Jul 14, 2026

Copy link
Copy Markdown

Fixes #69164

Updates the remaining stale SQLAlchemy 1.4 documentation link in \sql_alchemy_pool_pre_ping\ config to point at SQLAlchemy 2.0 docs.

  • File: \airflow-core/src/airflow/config_templates/config.yml`n- Change: \en/14/core/pooling\ -> \en/20/core/pooling`n- Doc/comment-only change; no behavior change

Important

🛠️ Maintainer triage note for @br413 · by @potiuk · 2026-07-18 15:46 UTC

Helpful heads-up from the maintainers — please address before this PR can be reviewed:

  • Pre-commit / static checks. See docs.

See the Pull Request quality criteria for details.

The ball is in your court — you've been assigned to this PR. Fix the above, then mark it Ready for review.

Automated triage — may be imperfect; a maintainer takes the next look.

@boring-cyborg

boring-cyborg Bot commented Jul 14, 2026

Copy link
Copy Markdown

Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide
Here are some useful points:

  • Pay attention to the quality of your code (ruff, mypy and type annotations). Our prek-hooks will help you with that.
  • In case of a new feature add useful documentation (in docstrings or in docs/ directory). Adding a new operator? Check this short guide Consider adding an example Dag that shows how users should use it.
  • Consider using Breeze environment for testing locally, it's a heavy docker but it ships with a working Airflow and a lot of integrations.
  • Be patient and persistent. It might take some time to get a review or get the final approval from Committers.
  • Please follow ASF Code of Conduct for all communication including (but not limited to) comments on Pull Requests, Mailing list and Slack.
  • Be sure to read the Airflow Coding style.
  • Always keep your Pull Requests rebased, otherwise your build might fail due to changes not related to your commits.
    Apache Airflow is a community-driven project and together we are making it better 🚀.
    In case of doubts contact the developers at:
    Mailing List: dev@airflow.apache.org
    Slack: https://s.apache.org/airflow-slack

@cursor
cursor Bot force-pushed the docs/sqlalchemy-2.0-pooling-link branch 2 times, most recently from 88ef2ca to de27b06 Compare July 19, 2026 10:21
@cursor
cursor Bot force-pushed the docs/sqlalchemy-2.0-pooling-link branch from de27b06 to 37a1682 Compare July 19, 2026 10:28
@br413 br413 changed the title docs: update SQLAlchemy pooling link from 1.4 to 2.0 Update SQLAlchemy pooling link from 1.4 to 2.0 Jul 19, 2026
@br413

br413 commented Jul 20, 2026

Copy link
Copy Markdown
Author

Closing this PR — the target link appears already updated on main. Will open a fresh contribution for a different Airflow docs issue.

@br413

br413 commented Jul 20, 2026

Copy link
Copy Markdown
Author

Superseded — no diff vs main for the original target. Contributor will open a new PR for a different issue.

@br413 br413 closed this Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

2 participants