diff options
author | Andy Schwerin <schwerin@mongodb.com> | 2017-06-28 17:13:47 -0400 |
---|---|---|
committer | Andy Schwerin <schwerin@mongodb.com> | 2017-07-21 10:53:56 -0400 |
commit | 01a38984d5c7b134b177d1b78023ed779c8b1631 (patch) | |
tree | 05c64a245c822b436cc07738a18212bb404963d8 | |
parent | aeccd2c8b429f4f3f81f5fb8e048ddb58fd0749a (diff) | |
download | mongo-01a38984d5c7b134b177d1b78023ed779c8b1631.tar.gz |
SERVER-29817 Use hinted insert when building chunkRangeMap
-rw-r--r-- | src/mongo/s/chunk_manager.cpp | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/src/mongo/s/chunk_manager.cpp b/src/mongo/s/chunk_manager.cpp index ecc743fd63f..70a79eee26f 100644 --- a/src/mongo/s/chunk_manager.cpp +++ b/src/mongo/s/chunk_manager.cpp @@ -383,14 +383,17 @@ ChunkManager::ChunkMapViews ChunkManager::_constructChunkMapViews(const OID& epo const BSONObj rangeMin = firstChunkInRange->getMin(); const BSONObj rangeMax = rangeLast->second->getMax(); - const auto insertResult = chunkRangeMap.insert(std::make_pair( - rangeMax, ShardAndChunkRange{{rangeMin, rangeMax}, firstChunkInRange->getShardId()})); + const auto oldSize = chunkRangeMap.size(); + const auto insertIterator = chunkRangeMap.insert( + chunkRangeMap.end(), + std::make_pair( + rangeMax, + ShardAndChunkRange{{rangeMin, rangeMax}, firstChunkInRange->getShardId()})); uassert(ErrorCodes::ConflictingOperationInProgress, str::stream() << "Metadata contains two chunks with the same max value " << rangeMax, - insertResult.second); + oldSize + 1 == chunkRangeMap.size()); - const auto& insertIterator = insertResult.first; if (insertIterator != chunkRangeMap.begin()) { // Make sure there are no gaps in the ranges |