[SERVER-80466] Convert ChangeLogType to be IDL Generated Created: 28/Aug/23 Updated: 31/Jan/24 Resolved: 30/Jan/24 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Mathis Bessa | Assignee: | Backlog - Service Architecture |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | ntdi_nice_to_have | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Assigned Teams: |
Service Arch
|
||||||||||||
| Participants: | |||||||||||||
| Description |
|
Currently the ChangeLogType is manually generated and does not include the serialization context logic when serializing / deserialzing. There is a TODO in the code base because we have to manually deserialize from a BSON object. If we were to use an IDL generator we would not have to do such work manually and potentially lose / drop a tenantId since we are not carrying a SerializationContext over. The NamespaceString in this case is constructed from BSON field values manually without a tenantId and without a serialization context which will be a problem once this code runs with multitenancy.
|
| Comments |
| Comment by Didier Nadeau [ 30/Jan/24 ] |
|
Won't do as we're focusing on tickets necessary for NTDI. |