[DRIVERS-2356] Reduce expected removeKeyAltName operations to a single findOneAndUpdate Created: 14/Jun/22 Updated: 07/Apr/23 Resolved: 07/Apr/23 |
|
| Status: | Closed |
| Project: | Drivers |
| Component/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Unknown |
| Reporter: | Ezra Chung | Assignee: | Ezra Chung |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Epic Link: | Key Management API | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Driver Changes: | Needed | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Quarter: | FY23Q2 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Downstream Changes Summary: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Driver Compliance: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Description |
Summary
The removeKeyAltName() function can be optimized to use only a single findOneAndUpdate() that still conditionally $unsets the keyAltNames field by using an aggregation pipeline. MotivationWho is the affected end user?Any Drivers that have begun implementing, or have already implemented, the new removeKeyAltName() function. How does this affect the end user?Drivers will have to update the unified spec test file(s) and implementation of removeKeyAltName() to satisfy the new expectations. How likely is it that this problem or use case will occur?N/A If the problem does occur, what are the consequences and how severe are they?N/A Is this issue urgent?No. This is an optimization. Is this ticket required by a downstream team?No. Is this ticket only for tests?No. The updated unified spec test will also require updating existing implementations of removeKeyAltName(). |
| Comments |
| Comment by Githook User [ 14/Jun/22 ] |
|
Author: {'name': 'Ezra Chung', 'email': '88335979+eramongodb@users.noreply.github.com', 'username': 'eramongodb'}Message: |