diff options
author | Alberto Lerner <alerner@10gen.com> | 2010-10-27 16:41:54 -0400 |
---|---|---|
committer | Alberto Lerner <alerner@10gen.com> | 2010-10-27 16:42:02 -0400 |
commit | d5985242144afdd8b6ea0d662d3c899a36dd0df5 (patch) | |
tree | 085429bc01334870749a2d378923abf93418f25f /s/balance.cpp | |
parent | 2998c9f313af005edec0c115bdc1ce8641b916ba (diff) | |
download | mongo-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.cpp | 10 |
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; |