[SERVER-37047] Move assertions out of testshard1.js that depend on data distribution Created: 07/Sep/18 Updated: 29/Oct/23 Resolved: 04/Mar/19 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Querying |
| Affects Version/s: | None |
| Fix Version/s: | 4.1.9 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Matthew Saltz (Inactive) | Assignee: | Charlie Swanson |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||
| Operating System: | ALL | ||||||||||||
| Participants: | |||||||||||||
| Linked BF Score: | 5 | ||||||||||||
| Description |
|
The balancer in testshard1.js is turned on to mimic a client enviroment where aggregations are issued concurrently with balancing. More recently the balancer was changed in a way that sometimes left the cluster with all chunks on one shard, which exposed some assertions in this test that depend on a particular data distribution. Such assertions are not appropriate for this test and should instead construct their own cluster which matches their assumptions. |
| Comments |
| Comment by Githook User [ 04/Mar/19 ] |
|
Author: {'name': 'Charlie Swanson', 'email': 'charlie.swanson@mongodb.com', 'username': 'cswanson310'}Message: Splits some assertions about explain out of the test to fix some build |
| Comment by Charlie Swanson [ 01/Mar/19 ] |
|
Since this is blocking a BF I did not close it, but rather adjusted the goal. I'll pick this up and do it as part of BF friday today. |
| Comment by Charlie Swanson [ 10/Sep/18 ] |
|
I think instead we should write the test to not rely on any particular distribution of data. In particular, any assertions that rely on data being spread across multiple shards should be re-written or moved to another test. |