-
Type:
Bug
-
Resolution: Done
-
Priority:
Blocker - P1
-
Affects Version/s: 2.8.0-rc3
-
Component/s: Performance, Write Ops
-
Fully Compatible
-
ALL
-
None
-
None
-
None
-
None
-
None
-
None
-
None
Problem
An observed 87% performance regression has been introduced by f59a05c611d66a03c1fb8579b54569bfc2971374
Last good commit the throughput is
1b88ce0b469d85cff0575a6b0a98daa31a3f44e0 @@@START@@@ Update.MmsIncDeep1 4 17675.90605543633 @@@END@@@
After this commit the throughput is
f59a05c611d66a03c1fb8579b54569bfc2971374 @@@START@@@ Update.MmsIncDeep1 4 2314.1996434616813 @@@END@@@
The git bisect for this is as follows
f59a05c611d66a03c1fb8579b54569bfc2971374 is the first bad commit
commit f59a05c611d66a03c1fb8579b54569bfc2971374
Author: Scott Hernandez <scotthernandez@gmail.com>
Date: Fri Dec 5 11:15:41 2014 -0500
SERVER-15622: check parents in dotted paths during validation:040000 040000 04cf9e7a4b79793f21d8d15234faa9e5d2abb45d fb62362f2ac1a8201d336393acc918c1b16f840d M jstests
:040000 040000 afc531a4380fa77458af2de2e64afafa527f7f1f aa6c6cb8e7fbd568426af6af9967d69b6bc16332 M src
Reproduce
var tests = [] load('mongo-perf/util/utils.js') var setupMMS = function( collection ) { collection.drop(); var base = { _id: 0, a: 0, h: {}, z: 0 }; for (var i = 0; i < 24; i++) { base.h[i] = {}; for (var j = 0; j < 60; j++) { base.h[i][j] = { n: 0, t: 0, v: 0 }; } } collection.insert(base); }; tests.push( { name: "Update.MmsIncDeep1", tags: ['update','sanity','mms','daily','weekly','monthly'], pre: setupMMS, ops: [ { op: "update", query: { _id: 0 }, update: { $inc: { "h.23.59.n": 1 } } } ] } ); runTests([4], 1, 8, 5, 1, 'test', 'sanity')