-
Notifications
You must be signed in to change notification settings - Fork 417
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
Show the correct amount of free memory available on the system #1179
Show the correct amount of free memory available on the system #1179
Conversation
Attempts to use the `available_memory` stat, otherwise tries to cobble it together using cached+buffered+free as per the erlang docs on `:memsup:get_system_memory_data` and only failing that flals back to `free_memory`
Co-authored-by: Jonatan Kłosko <[email protected]>
The CLA link is not working for me, btw. So I can't actually sign it :/ |
@aseigo can you please try again? And if it still fails, which error are you running into? |
It worked this time, @josevalim :) From the web console, it was getting errors from a github REST endpoint ... :/ But it works now! So, all's well that ends well? |
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.
Thanks!
Attempts to use the
available_memory
stat, otherwise tries to cobble it together using cached+buffered+free as per the erlang docs on:memsup:get_system_memory_data
and only failing that flals back tofree_memory
.I have only been able to test it so far on Linux, but have tested all the branches of the
measure_free_memory
functions. Testing it on other OS's would be good.Incorrect without this patch:
Correct, after this patch:
.