[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: |
|
||||
| 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:
|
| Comments |
| Comment by Githook User [ 05/Jul/18 ] |
|
Author: {'username': 'sulabhM', 'name': 'Sulabh Mahajan', 'email': 'sulabh.mahajan@mongodb.com'}Message: (cherry picked from commit 92ba01fe2c62ba716f9d0b05f23e2a2403539811) |
| Comment by Sulabh Mahajan [ 28/Jun/18 ] |
|
henrik.edin, |
| Comment by Githook User [ 28/Jun/18 ] |
|
Author: {'username': 'sulabhM', 'name': 'Sulabh Mahajan', 'email': 'sulabh.mahajan@mongodb.com'}Message: |
| 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 |
| 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: |