-
Notifications
You must be signed in to change notification settings - Fork 4
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
Memory leaks #9
Comments
Actually, I struggle to find why CowTable is leaking memory =( |
Hmm yeah I can definitely see the leak. How about I revert for now and you submit a new PR if/when you find the leak? |
Solved with switching to the This leaves us with garbage collection for the network itself. It grows when I add facts but never shrinks, independent of using CowTable or Table. |
Still leaking for me but I'll make a new release with the revert and we can take our time finding a fix. It's all good -- neither of us noticed :D |
When using pararules in my project, I noticed two kinds of memory leaks:
fireRules
.I'll fix the first as I introduced the cow solution, but I'm unsure what to do with the second.
The text was updated successfully, but these errors were encountered: