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

Test (and fix) memory leaks with logger #85

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

mariusandra
Copy link
Collaborator

@mariusandra mariusandra commented May 11, 2024

The logger seems to be running out of memory

  • Make tests to make sure this doesn't happen again.
  • Work in progress

@xTrayambak
Copy link

xTrayambak commented Oct 23, 2024

Are you sure this is occuring in your logging code? If you're using Nim 2.0, then you should know that Jester's underlying server (httpbeast) hasn't been updated for it and leaks memory under ORC (Nim's new memory management system). You would need to use jesterfork that uses httpbeastfork in order for it to work properly. I think Jester/Httpbeast's developer has sort of given up on Nim, he isn't very active with it anymore. I'd recommend you to check out prologue as well.

Source: We tried using Jester in prod and it started to leak memory in Nim 2 after it compiled after a bunch of manual patching

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.

2 participants