[SERVER-77590] Consolidate the validation paths for _id fields between inserts and updates Created: 30/May/23 Updated: 08/Jun/23 |
|
| Status: | Backlog |
| Project: | Core Server |
| Component/s: | Write Ops |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Chi-I Huang | Assignee: | Backlog - Query Optimization |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | tech-debt | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Assigned Teams: |
Query Optimization
|
||||||||||||
| Participants: | |||||||||||||
| Description |
|
There is a divergence of the validation path for _id fields between inserts & updates. We should refactor these two paths (and any additional create/update validation paths, if they exist) to rely on the same validation logic for consistency and increased test coverage. The validation is defined:
Background Originally, we were using storageValidEmbedded method in updates. The call stack was like:
Later in Due to this divergence, we had the issues |