[SERVER-75814] MDB compact commands running in testing aren't actually exercising WT compact because the data size is too small and WT skips it Created: 06/Apr/23 Updated: 04/Dec/23 |
|
| Status: | Open |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Dianna Hohensee (Inactive) | Assignee: | Backlog - Storage Engines Team |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Assigned Teams: |
Storage Engines
|
||||||||||||
| Sprint: | StorEng - Defined Pipeline, 2023-11-28 - Anthill Tiger | ||||||||||||
| Participants: | |||||||||||||
| Story Points: | 5 | ||||||||||||
| Description |
|
WT::compact skips compaction if the data size is too small
MDB layer JS tests calling the compact command aren't actually testing compaction because compaction is skipped due to too little data. For reference this is the WT::compact code. This is the list of effected tests: jstests/noPassthrough/index_version_autoupgrade.js. There are other tests call the MDB compact command, but the intent in those tests are things like auth, allowed to run on a secondary, view namespace handling, etc. |
| Comments |
| Comment by Sean Watt [ 29/Nov/23 ] |
|
I've taken a look through the jstests that use the compact command and many of them just test the usage of the server API and don't intend on compact doing any particular work. In response to the tests linked in the description:
A few other compact tests to consider are:
In summary the only tests worthwhile changing are
|
| Comment by Etienne Petrel [ 18/Jul/23 ] |
|
As keith.smith@mongodb.com said above, we have a set of tests dedicated to compact. However, with PM-3091, we would like MDB to introduce more testing in this area, especially with the new API that will be implemented. gregory.wlodarek@mongodb.com is the POC for those new tests. |