[GitHub] [airflow] xinbinhuang commented on a change in pull request #17600: Fix wrong Postgres search_path set up instructions

[GitBox <gi...@apache.org>]

xinbinhuang commented on a change in pull request #17600:
URL: https://github.com/apache/airflow/pull/17600#discussion_r693428091



##########
File path: docs/apache-airflow/howto/set-up-database.rst
##########
@@ -214,7 +214,7 @@ We recommend using the ``psycopg2`` driver and specifying it in your SqlAlchemy
    postgresql+psycopg2://<user>:<password>@<host>/<db>
 
 Also note that since SqlAlchemy does not expose a way to target a specific schema in the database URI, you may
-want to set a default schema for your role with a SQL statement similar to ``ALTER ROLE username SET search_path = airflow, foobar;``
+want to set a default schema for your role with a SQL statement similar to ``ALTER ROLE airflow_user SET search_path = airflow, public, utility;``

Review comment:
       ```suggestion
   want to set a default schema for your user with a SQL statement similar to ``ALTER USER airflow_user SET search_path = airflow, public, utility;``
   ```
   
   `ALTER ROLE` is alias to `ALTER USER`, so it'll be more consistent with the previous sections (i.e. `CREATE UESER ...`).
   
    




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