-
Notifications
You must be signed in to change notification settings - Fork 79
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
Conversation
…igners. Signed-off-by: Byron Gravenorst <[email protected]>
Signed-off-by: Byron Gravenorst <[email protected]>
Signed-off-by: Byron Gravenorst <[email protected]>
Signed-off-by: Byron Gravenorst <[email protected]>
…to bgravenorst-TEKU-194
Signed-off-by: Byron Gravenorst <[email protected]>
If you have more than 8GB of RAM, we recommend setting the maximum heap size to approximately | ||
5.632GB. | ||
|
There was a problem hiding this comment.
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...
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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). | |||
|
There was a problem hiding this comment.
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).
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Pull Request Description
Added a recommended value when using >8GB of RAM, and added a troubleshooting section.