[SERVER-34579] Do not populate indexDetails for mobile storage engine Created: 19/Apr/18  Updated: 29/Oct/23  Resolved: 28/Jun/18

Status: Closed
Project: Core Server
Component/s: Storage
Affects Version/s: None
Fix Version/s: 4.0.1, 4.1.1

Type: Task Priority: Major - P3
Reporter: Henrik Edin Assignee: Sulabh Mahajan
Resolution: Fixed Votes: 0
Labels: nonnyc, storage-engines
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Backports
Backwards Compatibility: Fully Compatible
Backport Requested:
v4.0
Sprint: Storage Non-NYC 2018-06-04, Storage Non-NYC 2018-06-18, Storage Non-NYC 2018-07-02
Participants:
Story Points: 5

 Description   

This causes apitest_dbcollection.js to fail on mongoe. Reproduce by:

python buildscripts\resmoke.py --suites=core --mongod=./mongoe jstests/core/apitest_dbcollection.js

2018-04-19T16:44:24.586-0400 E QUERY    [js] Error: [0] != [0] are equal : a_1 exists in indexDetails but contains no information: {
    "ns" : "test.apttest_dbcollection",
    "size" : 33,
    "count" : 1,
    "avgObjSize" : 33,
    "storageSize" : 33,
    "capped" : false,
    "nindexes" : 2,
    "indexDetails" : {
            "a_1" : {
            }
    },
    "totalIndexSize" : 22,
    "indexSizes" : {
            "_id_" : 16,
            "a_1" : 6
    },
    "ok" : 1
} :
doassert@src/mongo/shell/assert.js:18:14
assert.neq@src/mongo/shell/assert.js:207:9
checkIndexDetails@.\jstests\core\apitest_dbcollection.js:217:1
@.\jstests\core\apitest_dbcollection.js:226:1
@.\jstests\core\apitest_dbcollection.js:151:2



 Comments   
Comment by Githook User [ 05/Jul/18 ]

Author:

{'username': 'sulabhM', 'name': 'Sulabh Mahajan', 'email': 'sulabh.mahajan@mongodb.com'}

Message: SERVER-34579 Do NOT populate indexDetails for mobile SE

(cherry picked from commit 92ba01fe2c62ba716f9d0b05f23e2a2403539811)
Branch: v4.0
https://github.com/mongodb/mongo/commit/76d2b8d31566a58cef0f515ca1e32494bc67da5d

Comment by Sulabh Mahajan [ 28/Jun/18 ]

henrik.edin,
Thanks, I have removed the SERVER-34533 dependency on this ticket.

Comment by Githook User [ 28/Jun/18 ]

Author:

{'username': 'sulabhM', 'name': 'Sulabh Mahajan', 'email': 'sulabh.mahajan@mongodb.com'}

Message: SERVER-34579 Do NOT populate indexDetails for mobile SE
Branch: master
https://github.com/mongodb/mongo/commit/92ba01fe2c62ba716f9d0b05f23e2a2403539811

Comment by Henrik Edin [ 26/Jun/18 ]

sulabh.mahajan Sounds good!

Comment by Sulabh Mahajan [ 26/Jun/18 ]

henrik.edin, we will stop populating indexDetails for mobile SE with the fix for this ticket.

The test that hit this issue (jstests/core/apitest_dbcollection.js) was disabled along with disabling capped collections and incompatible_with_embedded_todo_investigate flag was removed. Likely SERVER-34533 doesn't depend on this ticket now.

Comment by Sulabh Mahajan [ 26/Jun/18 ]

Mobile SE doesn't have additional information on indexes to populate in the stats. This is similar to MMAP which doesn't populate indexDetails. The bug here is that we populate the indexDetails structure with the indexNames but nothing for the individual indexes. The change under review skips populating the indexDetail structure altogether for the mobile SE.

Comment by Sulabh Mahajan [ 19/Jun/18 ]

Status:
Haven't started working on it yet.

Generated at Thu Feb 08 04:37:09 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.