-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Fix: Increase MaxMemory param to use 1/4 of total memory. #9446
Conversation
Previously it was using 1/8 of the total memory. Signed-off-by: Antonio Navarro Perez <antnavper@gmail.com>
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.
@ajnavarro : looks good to me. I think it's also useful to give context in the PR where this is coming from: #9443 (comment)
This should also be added to #9442
@BigLep added a comment on the PR description. |
Per 2022-12-06 maintainer conversation, we're going to go from half to a quarter of system memory. |
Co-authored-by: Steve Loeppky <biglep@protocol.ai>
Per discussion here: #9443 (comment)
We are going to set the default max memory to 1/2 of the total memory instead of 1/8 to improve the default total inbound connections number, trying to avoid CM errors.
Signed-off-by: Antonio Navarro Perez antnavper@gmail.com