Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update the memory management values. #221

Merged
merged 16 commits into from
Jan 11, 2021
Merged

Update the memory management values. #221

merged 16 commits into from
Jan 11, 2021

Conversation

bgravenorst
Copy link
Contributor

Pull Request Description

Added a recommended value when using >8GB of RAM, and added a troubleshooting section.

Comment on lines 14 to 16
If you have more than 8GB of RAM, we recommend setting the maximum heap size to approximately
5.632GB.

Copy link
Contributor

@rolfyone rolfyone Dec 7, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if you have 8GB of ram, 5.5gb (-Xmx=5632m) works well.

If you have more than 8GB of ram, total memory - 4GB if the machine is only running a single instance of teku.
If the machine is used for multiple tasks, set java heap space to 6GB for teku, to limit it's memory usage to approximately 8GB.
that's my attempt, @ajsutton or @mbaxter would potentially have far more accurate answers...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd say,

If you have more than 8GB of RAM available, we recommend setting the maximum heap size to 5GB or more.

It's not an exact science, and half gigabytes are neither here nor there.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@@ -11,6 +11,9 @@ We recommend setting the maximum heap size to 2GB if you have less than 8GB of R
more RAM, but want to free up memory for other processes. This results in an approximate process
size of 3.5GB (on Linux).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd update these for mainnet, and definitely for Pyrmont.

We recommend setting the maximum heap size to at least 3GB if you have less than 8GB of RAM, or if you have more RAM, but want to free up memory for other processes. This results in an approximate process size of 5GB (on Linux).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also update the -Xmx2g to -Xmx3g in the examples below.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Signed-off-by: Byron Gravenorst <[email protected]>
Copy link
Contributor

@rolfyone rolfyone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bgravenorst bgravenorst merged commit 54f6be5 into Consensys:master Jan 11, 2021
@bgravenorst bgravenorst deleted the TEKU-16 branch January 11, 2021 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants