[CDRIVER-3477] _mongoc_bulk_operation_update_append does not update has_array_filters for subsequent update operations Created: 08/Jan/20 Updated: 28/Oct/23 Resolved: 18/May/20 |
|
| Status: | Closed |
| Project: | C Driver |
| Component/s: | Bulk API |
| Affects Version/s: | None |
| Fix Version/s: | 1.17.0-beta2, 1.17.0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Jeremy Mikola | Assignee: | Roberto Sanchez |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | low-hanging-fruits, new-eng | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||
| Description |
|
Assuming this was an oversight, I think the appropriate test would be to execute a bulk write with two updates against a <3.6 server when only the second update uses arrayFilters. The driver should raise a client-side error to conform with the CRUD specification. Rather than implement this as a prose test specific to the C driver, it may make sense to contribute it directly to the specifications repo so all drivers can benefit. |
| Comments |
| Comment by Githook User [ 25/May/20 ] |
|
Author: {'name': 'Roberto C. Sánchez', 'email': 'roberto@connexer.com', 'username': 'rcsanchez97'}Message: |
| Comment by Githook User [ 18/May/20 ] |
|
Author: {'name': 'Roberto C. Sánchez', 'email': 'roberto@connexer.com', 'username': 'rcsanchez97'}Message: |
| Comment by Roberto Sanchez [ 13/May/20 ] |
|
jmikola kevin.albertson, I have what I believe to be an appropriate spec test that captures this scenario. Should I write a SPEC and/or DRIVERS ticket, or just go straight to a PR in the specifications repo? |
| Comment by Roberto Sanchez [ 12/May/20 ] |
|
In order to successfully execute bulkWrite against a server <3.6, it is necessary that the CRUD test suite not require sessions, thus blocking on |