[SERVER-76864] Pass SerializationContext obj into CursorResponseBuilder Created: 04/May/23 Updated: 29/Oct/23 Resolved: 30/May/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 7.0.0-rc3 |
| 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-29, Server Serverless 2023-06-12 | ||||||||||||||||
| Participants: | |||||||||||||||||
| Story Points: | 3 | ||||||||||||||||
| Description |
|
CursorResponseBuilder::done() and CursorResponseBuilder::appendCursorResponseObject() build responses with a NamespaceString field that calls `NamespaceStringUtil::serialze()`, which will depend on a `SerializationContext` object to specify serialization behavior. We need to pass the `SerializationContext` object down the layers in order to have `done()` and `appendCursorResponseObject()` serialize correctly. 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 [ 26/May/23 ] |
|
Author: {'name': 'cortrain', 'email': '95658946+cortrain@users.noreply.github.com', 'username': 'cortrain'}Message: |