[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:
Depends
is depended on by PHPC-898 readConcern option should not be incl... Closed

 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:

Failed to parse: { getMore: 44504619209605, collection: "customers", readConcern: { level: "local" } }. Unrecognized field 'readConcern'.

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: PHPC-898: Bump libmongoc and libbson to 1.5.4

Includes CDRIVER-2003, which fixes an issue with queries using a read concern.
Branch: master
https://github.com/mongodb/mongo-php-driver/commit/faae013abeb38aa7d3e0407cbe9fab9e51d003e6

Comment by Githook User [ 30/Jan/17 ]

Author:

{u'username': u'jmikola', u'name': u'Jeremy Mikola', u'email': u'jmikola@gmail.com'}

Message: PHPC-898: Bump libmongoc and libbson to 1.5.4

Includes CDRIVER-2003, which fixes an issue with queries using a read concern.
Branch: v1.2
https://github.com/mongodb/mongo-php-driver/commit/faae013abeb38aa7d3e0407cbe9fab9e51d003e6

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: CDRIVER-2003 readConcern option should not be included in getMore commands
Branch: r1.5
https://github.com/mongodb/mongo-c-driver/commit/5e9fb16fe1fbde011ec7deda0df23c41c7dba172

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 'CDRIVER-2003-getmore-read-concern'

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 'CDRIVER-2003-getmore-read-concern'

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 'CDRIVER-2003-getmore-read-concern'

Comment by Githook User [ 27/Jan/17 ]

Author:

{u'username': u'bjori', u'name': u'Hannes Magnusson', u'email': u'bjori@php.net'}

Message: CDRIVER-2003 readConcern option should not be included in getMore commands
Branch: master
https://github.com/mongodb/mongo-c-driver/commit/1a1a065d054959b63b4d406d83978c701bf94207

Generated at Wed Feb 07 21:13:51 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.