You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2021/10/21 22:48:11 UTC

[GitHub] [druid] sergioferragut opened a new pull request #11827: docker mem reqs

sergioferragut opened a new pull request #11827:
URL: https://github.com/apache/druid/pull/11827


   Make docker memory requirements prominent in the tutorial.
   @techdocsmith 
   
   This PR has:
   - [x] been self-reviewed.
      - [ ] using the [concurrency checklist](https://github.com/apache/druid/blob/master/dev/code-review/concurrency.md) (Remove this item if the PR doesn't have any relation to concurrency.)
   - [ ] added documentation for new or modified features or behaviors.
   - [ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
   - [ ] added or updated version, license, or notice information in [licenses.yaml](https://github.com/apache/druid/blob/master/dev/license.md)
   - [ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
   - [ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for [code coverage](https://github.com/apache/druid/blob/master/dev/code-review/code-coverage.md) is met.
   - [ ] added integration tests.
   - [ ] been tested in a test Druid cluster.
   


-- 
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@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org


[GitHub] [druid] paul-rogers commented on a change in pull request #11827: docker mem reqs

Posted by GitBox <gi...@apache.org>.
paul-rogers commented on a change in pull request #11827:
URL: https://github.com/apache/druid/pull/11827#discussion_r734206970



##########
File path: docs/tutorials/docker.md
##########
@@ -32,6 +32,13 @@ This tutorial assumes you will download the required files from GitHub. The file
 
 * [Docker](https://www.docker.com/get-started)
 
+### Docker memory requirements
+
+The default `docker-compose.yml` launches eight containers: Zookeeper, PostgreSQL, and six Druid containers. Each Druid service is configured to use up to 7 GB of memory (6 GB direct memory and 1 GB heap). However, the Quickstart will not use all the available memory.
+
+Docker needs at least 6 GiB of memory available for the Druid cluster. If you experience any processes crashing with a 137 error code you likely don't have enough memory allocated to Docker.

Review comment:
       Thanks for the edit!
   
   It seems that [On native Linux, Docker can use all available host memory](https://stackoverflow.com/questions/62649255/how-to-increase-check-default-memory-docker-has-on-linux). So the memory limit seems to be a MacOS thing. It can be adjusted using the [Docker Desktop](https://docs.docker.com/desktop/mac/) for mac.
   
   This is worth noting because when we're on Linux (Ubuntu) we will be happy as clams, but then we get slammed when we do the same thing on MacOS.




-- 
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@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org


[GitHub] [druid] techdocsmith commented on a change in pull request #11827: docker mem reqs

Posted by GitBox <gi...@apache.org>.
techdocsmith commented on a change in pull request #11827:
URL: https://github.com/apache/druid/pull/11827#discussion_r734662100



##########
File path: docs/tutorials/docker.md
##########
@@ -32,6 +32,13 @@ This tutorial assumes you will download the required files from GitHub. The file
 
 * [Docker](https://www.docker.com/get-started)
 
+### Docker memory requirements
+
+The default `docker-compose.yml` launches eight containers: Zookeeper, PostgreSQL, and six Druid containers. Each Druid service is configured to use up to 7 GB of memory (6 GB direct memory and 1 GB heap). However, the Quickstart will not use all the available memory.
+
+Docker needs at least 6 GiB of memory available for the Druid cluster. If you experience any processes crashing with a 137 error code you likely don't have enough memory allocated to Docker.

Review comment:
       ```suggestion
   Docker needs at least 6 GiB of memory available for the Druid cluster. For Docker Desktop on Mac OS, adjust the memory settings in the [Docker Desktop preferences](https://docs.docker.com/desktop/mac/). If you experience a crash with a 137 error code you likely don't have enough memory allocated to Docker.
   ```
   Thanks for the idea @paul-rogers . 




-- 
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@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org


[GitHub] [druid] techdocsmith merged pull request #11827: docker mem reqs

Posted by GitBox <gi...@apache.org>.
techdocsmith merged pull request #11827:
URL: https://github.com/apache/druid/pull/11827


   


-- 
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@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org


[GitHub] [druid] techdocsmith commented on a change in pull request #11827: docker mem reqs

Posted by GitBox <gi...@apache.org>.
techdocsmith commented on a change in pull request #11827:
URL: https://github.com/apache/druid/pull/11827#discussion_r734662100



##########
File path: docs/tutorials/docker.md
##########
@@ -32,6 +32,13 @@ This tutorial assumes you will download the required files from GitHub. The file
 
 * [Docker](https://www.docker.com/get-started)
 
+### Docker memory requirements
+
+The default `docker-compose.yml` launches eight containers: Zookeeper, PostgreSQL, and six Druid containers. Each Druid service is configured to use up to 7 GB of memory (6 GB direct memory and 1 GB heap). However, the Quickstart will not use all the available memory.
+
+Docker needs at least 6 GiB of memory available for the Druid cluster. If you experience any processes crashing with a 137 error code you likely don't have enough memory allocated to Docker.

Review comment:
       ```suggestion
   Docker needs at least 6 GiB of memory available for the Druid cluster. If you're using Docker Desktop on Mac OS, adjust the memory settings in the [Docker Desktop preferences](https://docs.docker.com/desktop/mac/). If you experience a crash with a 137 error code you likely don't have enough memory allocated to Docker.
   ```
   Thanks for the idea @paul-rogers . 




-- 
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@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org