potiuk commented on a change in pull request #21666:
URL: https://github.com/apache/airflow/pull/21666#discussion_r810015790



##########
File path: docs/apache-airflow/installation/setting-up-the-database.rst
##########
@@ -22,7 +22,7 @@ Airflow requires a database. If you're just experimenting and 
learning Airflow,
 default SQLite option. If you don't want to use SQLite, then take a look at
 :doc:`/howto/set-up-database` to setup a different database.
 
-Usually, you need to run ``airflow db upgrade`` in order to create the 
database schema that Airflow can use.
+Usually, you need to run ``airflow db init`` in order to create the database 
schema that Airflow can use.

Review comment:
       Well. This is not entirely true. 
   
   * `airflow db upgrade` creates DB schema 
   * `airflow db init` does what upgrade does but it also adds default 
connections for example - which might be not expected/wanted
   
   I'd say we should describe both usages here.
   I've seen contradicting advices on that and while I agree "db upgrade" is 
not the best command to "install" airflow - it has been recommended as the way 
to start clean DB without the extra connections created. 
   
   Maybe we should change those commands to have better names in the future, 
but for now documenting the two usages is I think good idea.
   




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