support using RemoveAll($in) instead of Bulk. #32

Merged
merged 1 commit into from Apr 21, 2017

Conversation

Projects
None yet
2 participants
Owner

jameinel commented Apr 21, 2017

Mongo 2.4 doesn't support bulk operations anyway, but this still
allows us to do a little bit more batching instead of removing
exactly one-by-one.

Just introduces another Remover type and switches between them based on $out support.
That may not be the exactly right signal, but it is a close approximation that didn't make me
need to think about it significantly.

support using RemoveAll($in) instead of Bulk.
Mongo 2.4 doesn't support bulk operations anyway, but this still
allows us to do a little bit more batching instead of removing
exactly one-by-one.

mjs approved these changes Apr 21, 2017

LGTM

I'll have to do another mgopurge release once this has merged.

@jameinel jameinel merged commit 5c6f1c4 into juju:master Apr 21, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment