XD-DENG commented on a change in pull request #11109:
URL: https://github.com/apache/airflow/pull/11109#discussion_r493846671
##########
File path: docs/howto/initialize-database.rst
##########
@@ -23,9 +23,8 @@ Initializing a Database Backend
If you want to take a real test drive of Airflow, you should consider
setting up a real database backend and switching to the LocalExecutor.
-As Airflow was built to interact with its metadata using the great SqlAlchemy
-library, you should be able to use any database backend supported as a
-SqlAlchemy backend. We recommend using **MySQL** or **Postgres**.
+Airflow was built to interact with its metadata using SqlAlchemy
+with **MySQL** and **Postgres** as supported backends.
Review comment:
maybe not making perfect sense to me, correct me if I'm wrong @potiuk :
This page is meant for users who have already installed Airflow and tried
with SQLite (the earlier paragraph is `If you want to take a real test drive of
Airflow, you should consider setting up a real database backend and switching
to the LocalExecutor.`)
So I would like to suggest to apply your change + append with "`(SQLite is
used primarily for development purpose)`". In addition I think you missed the
"`as`" in your change ;-)
Sorry for being quite picky for documentation. It's quite essential to me
though.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
