[DRIVERS-2183] Support 'let' option in BulkWriteOptions Created: 31/Jan/22 Updated: 25/Oct/22 |
|
| Status: | Implementing |
| Project: | Drivers |
| Component/s: | CRUD |
| Fix Version/s: | None |
| Type: | Task | Priority: | Unknown |
| Reporter: | Julius Park (Inactive) | Assignee: | Julius Park (Inactive) |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | size-small, spec-change | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Driver Changes: | Needed | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Downstream Changes Summary: | Drivers must implement a new 'let' option in BulkWriteOptions. See ticket description for more info. Sync unified CRUD spec tests to 343a56fe0e8ac2a5e83a75170f9f3f149e5c7da9. |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Driver Compliance: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Description |
|
Drivers should add support for let to BulkWriteOptions. If specified, the option should be used to specify the top-level let option on all update and delete commands executed as part of that bulk write (the option does not apply to insert). Note that there a presently no plans to add let options to bulk write models (e.g. UpdateOneModel). Although update does support a c option on individual statements, delete does not (pending SERVER-63704). That work can be deferred for now. |
| Comments |
| Comment by Githook User [ 23/Feb/22 ] |
|
Author: {'name': 'Julius Park', 'email': 'Juliusgeo@gmail.com', 'username': 'juliusgeo'}Message: DRIVERS-2183 Support 'let' option in BulkWriteOptions (#1132) |
| Comment by Shane Harvey [ 17/Feb/22 ] |
| Comment by Jeremy Mikola [ 16/Feb/22 ] |
|
kateryna.kamenieva responded on HELP-31129, so I think we can move forward with this. For the time being, let can be added to BulkWriteOptions and will be used to specify the top-level let option for all update and delete commands executed as part of that bulk write (insert is excluded since the option does not apply there). We'll ignore statement-level c options for now, but that may be revisited down the line if we see user demand for it. I'll revise the issue title and description accordingly. |
| Comment by Jeremy Mikola [ 02/Feb/22 ] |
|
julius.park: See my comment in HELP-31129. I think this issue needs to be repurposed to add let to both BulkWriteOptions (where it will set let on each command issued as part of the bulk write) and individual write models (where it will set c options for individual statements). |