-
Notifications
You must be signed in to change notification settings - Fork 440
MongoDB - Add combined index #1283
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
Conversation
With multiple queues and thousands of messages, MongoDB is doing a full collection scan. The combined index solves that.
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.
Simple and effective. @makasim take a look please
Hey @makasim ! I see that you have triggered the CI but it failed somehow. I've checked it but seems that nothing is related to my changes. Not sure what to do next 🤷♂️ |
I could merge only if CI is green. Someone has to fix it. I won't do it. |
@makasim can you rerun the CI? |
@makasim can you please create a new tag? I would like to upgrade dependencies in Printify projects. Thanks in advance 🙏🏻 |
I have some problems with git subtree split on my machine. I'll try to cut some time to fix it. Feel free to ping me from time to time. |
I've recently migrated to mac m1 and it seems split-sh does not work on m1. I've tried to compile it at my machine, without luck so far. |
Hey there 👋
In Printify we are utilizing this library quite heavily.
Recently we had an incident with the super high CPU and I/O usage in MongoDB.
With multiple queues and thousands of messages per queue, MongoDB is doing a full collection scan.
The combined index solves that.