[SERVER-17328] Add class invariant that collection exists to MultiPlanStage/SubplanStage/CachedPlanStage Created: 19/Feb/15 Updated: 19/Sep/15 Resolved: 14/Apr/15 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Querying |
| Affects Version/s: | None |
| Fix Version/s: | 3.1.2 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | J Rassi | Assignee: | David Storch |
| Resolution: | Done | Votes: | 0 |
| Labels: | neweng | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Backwards Compatibility: | Fully Compatible |
| Operating System: | ALL |
| Sprint: | Quint Iteration 3.1.2 |
| Participants: |
| Description |
|
MultiPlanStage/SubplanStage/CachedPlanStage are collection-aware stages (they interact with the plan cache, among other things), and they are never used with a non-existent collection. It would simplify the logic in these stages to be able to assume that the collection exists. invariant(collection) should be added to the constructor for these stages, and the header files should be updated as such. |
| Comments |
| Comment by Githook User [ 14/Apr/15 ] |
|
Author: {u'username': u'dstorch', u'name': u'David Storch', u'email': u'david.storch@10gen.com'}Message: |