uranusjr commented on a change in pull request #19053: URL: https://github.com/apache/airflow/pull/19053#discussion_r732226978
########## File path: docs/apache-airflow/tutorial.rst ########## @@ -374,9 +374,30 @@ Lets look at another example; we need to get some data from a file which is host Initial setup '''''''''''''''''''' We need to have docker and postgres installed. -We will be using this `docker file <https://airflow.apache.org/docs/apache-airflow/stable/start/docker.html#docker-compose-yaml>`_ +We will be using this :doc:`docker file <start/docker>` Follow the instructions properly to set up Airflow. +.. note:: + + Airflow manages databases using :doc:`connections <howto/connection>`. + When running the code below, you will need to either configure a relevant postgres connection named ``postgres_docker`` or configure one + as appropriate for your system and update the code below to reference the appropriate Connection id. + + If using the aforementioned :doc:`docker setup <start/docker>` + note that none of the out-of-the-box connections are quite right; you will likely (either via the webserver UI or command line) + want to add a docker-appropriate postgres connection (the following creates one that matches postgres as + configured in ``docker-compose.yml``): + +.. code-block:: bash Review comment: > it was easier to debug other syntax issues with it separate What's wrong with ```rst .. note:: Airflow manages databases using ... .. code-block:: bash airflow connections ... ``` ? -- 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]
