[CDRIVER-2003] readConcern option should not be included in getMore commands Created: 25/Jan/17 Updated: 31/Jan/17 Resolved: 27/Jan/17 |
|
| Status: | Closed |
| Project: | C Driver |
| Component/s: | libmongoc |
| Affects Version/s: | 1.5.3 |
| Fix Version/s: | 1.5.4 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Jeremy Mikola | Assignee: | Hannes Magnusson |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Description |
|
In mongodb/mongo-php-driver#516, a user encountered the following error when iterating on a large result set on a find operation that included a "local" read concern level:
If we look at _mongoc_cursor_cursorid_get_more(), it calls _mongoc_cursor_prepare_getmore_command() to create the getMore command document. That document is then passed along to _mongoc_cursor_cursorid_refresh_from_command() and executed with _mongoc_cursor_run_command(), wherein a readConcern field may be appended. The logic for appending the readConcern needs to be a bit more restrictive so that it only triggers for supported commands. |
| Comments |
| Comment by Githook User [ 30/Jan/17 ] |
|
Author: {u'username': u'jmikola', u'name': u'Jeremy Mikola', u'email': u'jmikola@gmail.com'}Message: Includes |
| Comment by Githook User [ 30/Jan/17 ] |
|
Author: {u'username': u'jmikola', u'name': u'Jeremy Mikola', u'email': u'jmikola@gmail.com'}Message: Includes |
| Comment by Githook User [ 28/Jan/17 ] |
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: Merge branch 'master' of github.com:mongodb/mongo-c-driver
|
| Comment by Githook User [ 27/Jan/17 ] |
|
Author: {u'username': u'bjori', u'name': u'Hannes Magnusson', u'email': u'bjori@php.net'}Message: |
| Comment by Githook User [ 27/Jan/17 ] |
|
Author: {u'username': u'bjori', u'name': u'Hannes Magnusson', u'email': u'bjori@php.net'}Message: Merge branch '
|
| Comment by Githook User [ 27/Jan/17 ] |
|
Author: {u'username': u'bjori', u'name': u'Hannes Magnusson', u'email': u'bjori@php.net'}Message: Merge branch '
|
| Comment by Githook User [ 27/Jan/17 ] |
|
Author: {u'username': u'bjori', u'name': u'Hannes Magnusson', u'email': u'bjori@php.net'}Message: Merge branch '
|
| Comment by Githook User [ 27/Jan/17 ] |
|
Author: {u'username': u'bjori', u'name': u'Hannes Magnusson', u'email': u'bjori@php.net'}Message: |