[SERVER-33541] Add snapshot read support for aggregation Created: 28/Feb/18 Updated: 29/Oct/23 Resolved: 09/Mar/18 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Storage |
| Affects Version/s: | None |
| Fix Version/s: | 3.7.3 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Eric Milkie | Assignee: | David Storch |
| 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: | Query 2018-03-12, Query 2018-03-26 | ||||||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||||||
| Description |
|
| Comments |
| Comment by Githook User [ 09/Mar/18 ] |
|
Author: {'email': 'david.storch@10gen.com', 'name': 'David Storch', 'username': 'dstorch'}Message: |
| Comment by David Storch [ 05/Mar/18 ] |
Filed |
| Comment by David Storch [ 01/Mar/18 ] |
|
Recording answers to some of my open questions after discussing with james.wahlin and milkie:
Yes, we should do so, in addition to prohibiting the $out stage. We also need to prohibit change streams, and probably $geoNear.
No specific problems other than the one below, though agree that we should have a targeted test case for $lookup.
This is something which we will probably fix. Dave will confirm that this issue exists and file a ticket as part of the local snapshot reads project.
Not that we could think of. Eric points out that the lock acquisitions in the agg path need to be updated to have a timeout of zero, but that this work can happen in a later sweep. |