[SERVER-76612] Copy SerializationContext between requests for views on agg Created: 27/Apr/23 Updated: 29/Oct/23 Resolved: 04/May/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 7.1.0-rc0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Hugh Tong (Inactive) | Assignee: | Hugh Tong (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | ntdi_must_have | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||
| Sprint: | Server Serverless 2023-05-15 | ||||||||
| Participants: | |||||||||
| Story Points: | 1 | ||||||||
| Description |
|
When aggregation has a view, it will attempt to resolve that view onto the base collection, and create a new agg command request based on the resolved namespace. However, this loses the serialization flags on the existing request that can cause serialization issues, so we need to copy the serialization context object over to the new request at the point of creation. As the goal of these changes are in pursuit of fixing tests that will fail once serializeForCommands and deserializeForCommands are plumbed in, no new tests are being added for this ticket. |
| Comments |
| Comment by Githook User [ 04/May/23 ] |
|
Author: {'name': 'Hugh Tong', 'email': 'hugh.tong@mongodb.com', 'username': 'cortrain'}Message: |
| Comment by Hugh Tong (Inactive) [ 28/Apr/23 ] |
|
This is a simple change, but the investigation that lead to this discovery is not being reflected in the story points. |