-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed #6
Comments
I think there is a memory leak in mqemitter-mongodb under high load, and it lives here: https://github.com/mcollina/mqemitter-mongodb/blob/798d191ce36e6ac8dac39a3b8d1713aba632bba2/mqemitter-mongodb.js#L170-L215. |
I think that error could cause some message lost but not what I’m sepaking about, am I wrong?
…---
Daniel - Software Engineer
Support me at:
Github sponsors
On 29 Apr 2020, at 12:26, Matteo Collina ***@***.***> wrote:
I think there is a memory leak in mqemitter-mongodb under high load, and it lives here: https://github.com/mcollina/mqemitter-mongodb/blob/798d191ce36e6ac8dac39a3b8d1713aba632bba2/mqemitter-mongodb.js#L170-L215.
Last time I checked, I was not able to track it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
I think whenever a message is lost it leaks memory. |
Are you speaking about this line? https://github.com/mcollina/mqemitter-mongodb/blob/798d191ce36e6ac8dac39a3b8d1713aba632bba2/mqemitter-mongodb.js#L213 |
Could we instead of using mongodb object id use a simple incremental id? |
I don't think it would change the problem. I don't think that callback is called when a message is missed. |
This error afetr 1 minute running a benchmark against aedes with mongodb backend (it happens both with and without clusters). Thoughts @mcollina ?
Seems the bottle-neck is mongodb
insert
method.If I use aedes-persistence-mongodb with mqemitter-redis everything works fine and performances are even better then using only redis persitence.
It even works with mqemitter-mongodb paired with aedes-persistence-redis but with worse performances
I think mqemitter-mongodb and aedes-persistence mongodb needs some thoughts about a way to optimize inserts in order to fix this
The text was updated successfully, but these errors were encountered: