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

bulk divide into actions, closes #265 #297

Closed
wants to merge 5 commits into from
Closed

Conversation

lionvs
Copy link
Contributor

@lionvs lionvs commented Mar 19, 2016

This is manual fix for issue #265

If group of bulk operations have more than 1000 operations, it divides the group into smaller groups of 1000 or less.

This work-around needs to be removed when it will be implemented in mongodb driver.

@chriswininger
Copy link

Be nice to see this pulled in. I just updated to 2.3.0 and immediately started running into this

@saintedlama
Copy link
Collaborator

@sorribas @watson @mafintosh What do you think about it? Could not find the reason for this issue but the fix would work. Merge it as temporary workaround?

@aPoCoMiLogin
Copy link

Any reason why this is not merged yet ?

@vctfernandes
Copy link

Any reason why this is not merged yet ? [2]

@saintedlama
Copy link
Collaborator

Implemented in commit 8c301a3 by adding conditions that open a new bulk cmd if the current documents/updates/deletes length exceeds the max bulk size. This implementation avoids copying possibly large bulks.

@saintedlama saintedlama closed this Aug 8, 2017
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.

5 participants