[CXX-97] C++ driver should allow GridFS slaveOk querying Created: 11/Apr/12 Updated: 08/Jul/14 Resolved: 02/Jul/14 |
|
| Status: | Closed |
| Project: | C++ Driver |
| Component/s: | API |
| Affects Version/s: | None |
| Fix Version/s: | legacy-0.11.0 |
| Type: | New Feature | Priority: | Major - P3 |
| Reporter: | Aristarkh Zagorodnikov | Assignee: | Tyler Brock |
| Resolution: | Done | Votes: | 1 |
| Labels: | cxxmove, gridfs, legacy-cxx, neweng, replicaset, slaveOk | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
Currently, C++ driver GridFS API does not support slaveOk. |
| Comments |
| Comment by Tyler Brock [ 02/Jul/14 ] |
|
merged |
| Comment by Githook User [ 02/Jul/14 ] |
|
Author: {u'username': u'TylerBrock', u'name': u'Tyler Brock', u'email': u'tyler.brock@gmail.com'}Message: |
| Comment by Tyler Brock [ 02/Jul/14 ] |
| Comment by Andrew Morrow (Inactive) [ 02/Jul/14 ] |
|
Hi onyxmaster - RE async: yes, we are thinking about it, but we do not have concrete plans at this point. Thanks, |
| Comment by Aristarkh Zagorodnikov [ 02/Jul/14 ] |
|
Thanks for the response. I look forward to new C++ driver that would support all the bells and whistles for GridFS queries. As a side note, I wonder if there is an idea to have an async (epoll, etc.)-based driver. Currently nginx-gridfs module works too slow only because there's no async support. While rolling one's own "driver" (it only needs to support a subset of the protocol, only predetermined queries) is perfectly feasible for such specific case, I'm unsure about what plans do you have on this matter. |
| Comment by Tyler Brock [ 30/Jun/14 ] |
|
I know it has been a while onyxmaster, but we are looking at this now and want to keep you updated. Thank you for your patience. We definitely need to allow for secondary reads for GridFS but doesn't make sense for GridFile::write to do so as secondaries do not take writes. Since this pull request and change was proposed we have forked the C++ driver from the server and are working on adding improvements such as this one to the driver. I'm going to try to add what feels like the same functionality but with ReadPreference (as that is how we do things now). |
| Comment by Aristarkh Zagorodnikov [ 20/Apr/12 ] |
|
I understand that, so I worked around by adding a source patch in https://bitbucket.org/onyxmaster/mod_gridfs =) |
| Comment by Eliot Horowitz (Inactive) [ 20/Apr/12 ] |
|
We are definitely planning on doing this - just have a lot of things to get done. |
| Comment by Aristarkh Zagorodnikov [ 19/Apr/12 ] |
|
On the other hand, this still might be useful for other applications if you intend to release C++ driver separately. |
| Comment by Aristarkh Zagorodnikov [ 19/Apr/12 ] |
|
Well, it looks like I have to roll my own driver for mod_gridfs, feel free to close this issue. |
| Comment by Aristarkh Zagorodnikov [ 15/Apr/12 ] |
|
Improved pull request a little bit to preserve parameter order. |
| Comment by Aristarkh Zagorodnikov [ 12/Apr/12 ] |
|
I've added a pull request that implements this feature: https://github.com/mongodb/mongo/pull/210. |