[GitHub] [airflow] uranusjr commented on a change in pull request #18484: Docs: Fix grammar in ``docs/apache-airflow/start/docker.rst``

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

uranusjr commented on a change in pull request #18484:
URL: https://github.com/apache/airflow/pull/18484#discussion_r715191362



##########
File path: docs/apache-airflow/start/docker.rst
##########
@@ -49,8 +49,8 @@ Older versions of ``docker-compose`` do not support all the features required by
 
 .. warning::
     Default amount of memory available for Docker on MacOS is often not enough to get Airflow up and running.
-    If you have not enough memory available it might lead to airflow webserver continuously restarting.
-    You should have at least 4GB memory allocated for the Docker Engine (ideally 8GB). You can check
+    If enough memory is not allocated, it might lead to airflow webserver continuously restarting.
+    You should at least allocate 4GB memory for the Docker Engine (ideally 8GB). You can check

Review comment:
       FWIW I’ve been running Breeze with 3.9GB RAM (I allocate 4GB to WSL2 and the host system uses a little bit). Occassionally the webserver fails to allocate memory when I run `start-airflow` but generally works fine (the webserver usually works on second try).




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