deepujain commented on code in PR #69150:
URL: https://github.com/apache/airflow/pull/69150#discussion_r3746794348


##########
airflow-core/docs/howto/set-up-database.rst:
##########
@@ -276,6 +276,13 @@ If you use a current Postgres user with custom 
search_path, search_path can be c
 
 For more information regarding setup of the PostgreSQL connection, see 
`PostgreSQL dialect 
<https://docs.sqlalchemy.org/en/20/dialects/postgresql.html>`__ in SQLAlchemy 
documentation.
 
+.. note::
+
+   PostgreSQL connection strings can include multiple fallback hosts. Use 
SQLAlchemy's documented PostgreSQL
+   URL format for this setup so each fallback host is passed correctly to the 
selected PostgreSQL driver. See
+   `Specifying multiple fallback hosts 
<https://docs.sqlalchemy.org/en/20/dialects/postgresql.html#specifying-multiple-fallback-hosts>`__
+   in the SQLAlchemy documentation.

Review Comment:
   Updated the note to scope this to SQLAlchemy's psycopg2, psycopg, and 
asyncpg dialects. It now calls out driver-specific port requirements and links 
both the fallback-host and asyncpg multihost sections. The scoped pre-commit 
check and `git diff --check` pass.
   
   ---
   Drafted-by: OpenAI Codex (no human review before posting)



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