vishalmore90 opened a new pull request, #69535:
URL: https://github.com/apache/airflow/pull/69535

   ### Context
   
   PR #69200 updated several SQLAlchemy documentation links from `en/14` 
(SQLAlchemy 1.4) to `en/20` (SQLAlchemy 2.0), reflecting that Airflow core now 
requires SQLAlchemy 2.0+.
   However, that PR:
   1. Missed updating the pooling documentation link in `config.yml`.
   2. Incorrectly updated a historical 1.4 changelog link in 
`set-up-database.rst` (where the `postgres://` scheme deprecation was 
introduced) to point to the `en/20` docs.
   3. Left behind outdated warning prose referring to "SQLAlchemy 1.4+" in the 
database setup guide, which is no longer relevant now that Airflow 3 requires 
SQLAlchemy 2.0+ exclusively.
   
   This PR addresses these omissions and the feedback from issue #69164.
   
   ### Verification
   
   - Run `breeze build-docs` (Docker environment not available locally, but the 
syntax, links, and anchors have been manually verified to resolve correctly).
   - Verified target URLs:
     - Reverted link: 
`https://docs.sqlalchemy.org/en/14/changelog/changelog_14.html#change-3687655465c25a39b968b4f5f6e9170b`
     - Updated link: 
`https://docs.sqlalchemy.org/en/20/core/pooling.html#disconnect-handling-pessimistic`
   
   ### Checklist
   
   - [x] Link to the issues/PRs: Closes #69164
   - [x] Documentation-only change (no newsfragment needed)
   


-- 
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]

Reply via email to