summaryrefslogtreecommitdiff
path: root/s/balance.cpp
diff options
context:
space:
mode:
authorAlberto Lerner <alerner@10gen.com>2010-10-27 16:41:54 -0400
committerAlberto Lerner <alerner@10gen.com>2010-10-27 16:42:02 -0400
commitd5985242144afdd8b6ea0d662d3c899a36dd0df5 (patch)
tree085429bc01334870749a2d378923abf93418f25f /s/balance.cpp
parent2998c9f313af005edec0c115bdc1ce8641b916ba (diff)
downloadmongo-d5985242144afdd8b6ea0d662d3c899a36dd0df5.tar.gz
don't split after a balance move fails (a lock taken would cause it) and defer splitting jumbos to SERVER-1358
Diffstat (limited to 's/balance.cpp')
-rw-r--r--s/balance.cpp10
1 files changed, 3 insertions, 7 deletions
diff --git a/s/balance.cpp b/s/balance.cpp
index db8cb9eb3b7..9d6391492fa 100644
--- a/s/balance.cpp
+++ b/s/balance.cpp
@@ -73,13 +73,9 @@ namespace mongo {
continue;
}
- log() << "MOVE FAILED **** " << res << "\n"
- << " from: " << chunkInfo.from << " to: " << chunkInfo.to << " chunk: " << chunkToMove << endl;
-
- if ( res["split"].trueValue() ) {
- log() << "move asked for a split of " << c << endl;
- c->singleSplit( true /* force a split even if not enough data */ );
- }
+ // the move requires acquiring the collection metadata's lock, which can fail
+ log() << "balacer move failed: " << res << " from: " << chunkInfo.from << " to: " << chunkInfo.to
+ << " chunk: " << chunkToMove << endl;
}
return movedCount;