[PR] Do not warn about bare postgresql:// metadata DB URLs [airflow]

["jedcunningham (via GitHub)" <gi...@apache.org>]

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

   Airflow emitted a `FutureWarning` ("Bad scheme ... You must change to `postgresql+psycopg`") for bare `postgresql://` metadata DB URLs. Those URLs are valid: Airflow only rewrites them to make the driver explicit (`postgresql+psycopg` if psycopg 3 is installed, otherwise `postgresql+psycopg2`). Users read the warning as "driver-less URLs are deprecated".
   
   This change:
   - stops warning for bare `postgresql://` (the rewrite itself is unchanged);
   - keeps the warning for the actually unsupported `postgres://` and `postgres+psycopg2://` schemes, reworded to say what Airflow does and that `postgresql` (driver picked by Airflow) is a valid fix; the outdated SQLAlchemy 1.4 / Airflow 2.3 reference and the "before the next Airflow release" deadline are dropped;
   - updates the `_upgrade_postgres_metastore_conn` docstring accordingly;
   - adds a test that the warning is emitted only for `postgres` / `postgres+psycopg2`, and not for bare `postgresql`.
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Do not warn about bare postgresql:// metadata DB URLs [airflow]

["github-actions[bot] (via GitHub)" <gi...@apache.org>]

github-actions[bot] commented on PR #74863:
URL: https://github.com/apache/airflow/pull/74863#issuecomment-6220843094

   ### Backport successfully created: v3-4-test
   
   Note: As of [Merging PRs targeted for Airflow 3.X](https://github.com/apache/airflow/blob/main/dev/README_AIRFLOW3_DEV.md#merging-prs-targeted-for-airflow-3x)
   the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.
   
   In matter of doubt please ask in [#release-management](https://apache-airflow.slack.com/archives/C03G9H97MM2) Slack channel.
   
   <table>
           <tr>
           <th>Status</th>
           <th>Branch</th>
           <th>Result</th>
       </tr>
       <tr>
           <td>✅</td>
           <td>v3-4-test</td>
           <td><a href="https://github.com/apache/airflow/pull/74884"><img src="https://img.shields.io/badge/PR-74884-blue" alt="PR Link"></a></td>
       </tr>
   </table>


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Do not warn about bare postgresql:// metadata DB URLs [airflow]

["jedcunningham (via GitHub)" <gi...@apache.org>]

jedcunningham merged PR #74863:
URL: https://github.com/apache/airflow/pull/74863


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org