[CDRIVER-523] mongoc doesn't coalesce updates or deletes, only inserts Created: 05/Feb/15 Updated: 08/Jan/24 Resolved: 18/May/15 |
|
| Status: | Closed |
| Project: | C Driver |
| Component/s: | Bulk API, libmongoc |
| Affects Version/s: | None |
| Fix Version/s: | 1.1.6 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Hannes Magnusson | Assignee: | A. Jesse Jiryu Davis |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||
| Description |
|
When creating a batch of writes using the bulk interface, only inserts are coalesced, resulting in far many updates and delete roundtrips then expected.
Results in 1 round trip.
Results in 3 round trip.
Results in 3 round trip. |
| Comments |
| Comment by Githook User [ 27/May/15 ] |
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: |
| Comment by Githook User [ 18/May/15 ] |
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: |
| Comment by Githook User [ 18/May/15 ] |
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: |
| Comment by Githook User [ 18/May/15 ] |
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: new leak in write commands From the |
| Comment by Githook User [ 18/May/15 ] |
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: |
| Comment by Githook User [ 18/May/15 ] |
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: |
| Comment by Githook User [ 18/May/15 ] |
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: |
| Comment by A. Jesse Jiryu Davis [ 08/May/15 ] |
| Comment by A. Jesse Jiryu Davis [ 20/Apr/15 ] |
|
Right, that would be my plan. |
| Comment by Hannes Magnusson [ 20/Apr/15 ] |
|
Ok. If its OK to include this between the 1.2.0 beta<->final then I'm all good |
| Comment by A. Jesse Jiryu Davis [ 20/Apr/15 ] |
|
Thanks Hannes! 1.2 beta is due by the end of the month, if getting this correctly merged into 1.2 by then isn't practical, that's ok, it's just a performance enhancement. |
| Comment by Hannes Magnusson [ 20/Apr/15 ] |
|
I started couple of weeks ago - there are bunch of annoying conflicts for the 1.2 branch that needs careful verification. jesse What is the timetable for 1.2.0? |
| Comment by A. Jesse Jiryu Davis [ 20/Apr/15 ] |
|
bjori would you like to do the merge of your pull requests to master? |
| Comment by Hannes Magnusson [ 02/Apr/15 ] |
|
jesse Are you able to merge the patches (or alternatively reject them)? |
| Comment by Hannes Magnusson [ 23/Mar/15 ] |
|
mira.carey@mongodb.com I think this fell between the cracks... afaict the PRs are ready to be pulled? |
| Comment by Hannes Magnusson [ 05/Feb/15 ] |
|
Turns out there are already outstanding Pull Requests for this. |