[SERVER-24771] Make queries on views return a cursor on that view Created: 23/Jun/16 Updated: 02/Sep/16 Resolved: 26/Aug/16 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Querying |
| Affects Version/s: | None |
| Fix Version/s: | 3.3.12 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Geert Bosch | Assignee: | James Wahlin |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Backwards Compatibility: | Major Change | ||||||||
| Sprint: | Integration 18 (08/05/16), Integration 2016-08-29, Integration 2016-09-19 | ||||||||
| Participants: | |||||||||
| Linked BF Score: | 0 | ||||||||
| Description |
|
Queries on views should not return a cursor on the underlying collection, as that would result in problems with getMore if clients don't use the returned name space. Moreover, this may cause problems with auth or with killCursors. |
| Comments |
| Comment by Githook User [ 29/Aug/16 ] | ||||||||||||||||||||||||||||||||
|
Author: {u'username': u'jameswahlin', u'name': u'James Wahlin', u'email': u'james.wahlin@10gen.com'}Message: | ||||||||||||||||||||||||||||||||
| Comment by Githook User [ 26/Aug/16 ] | ||||||||||||||||||||||||||||||||
|
Author: {u'username': u'jameswahlin', u'name': u'James Wahlin', u'email': u'james.wahlin@10gen.com'}Message: | ||||||||||||||||||||||||||||||||
| Comment by Githook User [ 26/Aug/16 ] | ||||||||||||||||||||||||||||||||
|
Author: {u'username': u'jameswahlin', u'name': u'James Wahlin', u'email': u'james.wahlin@10gen.com'}Message: | ||||||||||||||||||||||||||||||||
| Comment by Githook User [ 26/Aug/16 ] | ||||||||||||||||||||||||||||||||
|
Author: {u'username': u'jameswahlin', u'name': u'James Wahlin', u'email': u'james.wahlin@10gen.com'}Message: | ||||||||||||||||||||||||||||||||
| Comment by Githook User [ 22/Aug/16 ] | ||||||||||||||||||||||||||||||||
|
Author: {u'username': u'GeertBosch', u'name': u'Geert Bosch', u'email': u'geert@mongodb.com'}Message: | ||||||||||||||||||||||||||||||||
| Comment by Kyle Suarez [ 17/Aug/16 ] | ||||||||||||||||||||||||||||||||
|
This should do for now:
Ideally, I would like this to go in the auth commands lib, but currently it cannot handle testing an actual cursor. The current expectsFail in that test code is also insufficient because it doesn't allow us to specify a specific error code, so we can't test that it doesn't fail with ErrorCodes.CommandNotSupportedOnView. In any case, I have | ||||||||||||||||||||||||||||||||
| Comment by Geert Bosch [ 17/Aug/16 ] | ||||||||||||||||||||||||||||||||
|
kyle.suarez, if you already have a test case for this (maybe as part of |