[SERVER-60071] Move clustered index jstests out of the timeseries folder Created: 20/Sep/21 Updated: 29/Oct/23 Resolved: 06/Oct/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 5.2.0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Josef Ahmad | Assignee: | Josef Ahmad |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||
| Sprint: | Execution Team 2021-10-04, Execution Team 2021-10-18 | ||||||||||||
| Participants: | |||||||||||||
| Description |
|
Some of the tests in jstests/core/timeseries/clustered_index_types.js cover generic clustered index features that are beyond the domain of time series collections. Move these tests to a dedicated folder in jstests/core, and generalise them to use the public clustered indexes API. |
| Comments |
| Comment by Githook User [ 06/Oct/21 ] |
|
Author: {'name': 'Josef Ahmad', 'email': 'josef.ahmad@mongodb.com', 'username': 'josefahmad'}Message: |
| Comment by Haley Connelly [ 05/Oct/21 ] |
|
josef.ahmad should be good to go now! |
| Comment by Haley Connelly [ 29/Sep/21 ] |
|
I'd propose perhaps renaming the jstests/core/timeseries/clustered_index_types.js test also. It doesn't necessarily test clusteredIndex 'types', it appears to be more the allowed crud ops on a clustered time series buckets collection. I think the other timeseries/clustered* jstests serve as coverage specific to timeseries we may want to keep. Definitely worth an investigation! I agree, the naming can get confusing between what files are in the timeseries directory versus the general core directory |
| Comment by Louis Williams [ 20/Sep/21 ] |
|
We don't necessarily need a folder. We just need to take the test out of the time-series directory. |