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

run gc after each request #4

Merged
merged 1 commit into from Jun 24, 2017

Conversation

Projects
None yet
2 participants
@grunxen
Copy link
Contributor

grunxen commented Jun 24, 2017

Run garbage collector after each server request for decreasing memory usage.

@castwide castwide merged commit f218fd1 into castwide:master Jun 24, 2017

@castwide

This comment has been minimized.

Copy link
Owner

castwide commented Jun 24, 2017

Thanks for the contribution. Is memory usage a major issue on your setup?

The GC call seemed to have minimal impact for me (consistently less than 10% difference), but it didn't appear to have any detrimental side effects either, so I went ahead and merged it.

@grunxen

This comment has been minimized.

Copy link
Contributor

grunxen commented Jun 24, 2017

Is memory usage a major issue on your setup?

Yes. Without gc calls it could use ~800mb of memory, now ~250mb.

@castwide

This comment has been minimized.

Copy link
Owner

castwide commented Jun 24, 2017

Wow, that's huge. Thanks again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment