Skip to content

Update parameters-map-reduce.rst #1003

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

Update parameters-map-reduce.rst #1003

wants to merge 2 commits into from

Conversation

wshaver
Copy link

@wshaver wshaver commented May 17, 2013

The sort key must be in an existing index for this collection.

Will Shaver added 2 commits May 17, 2013 11:32
Update parameters-map-reduce.rst
9e331a1 
The sort key must be in an existing index for this collection.
explain that reduce may not be called at all
5add70b
wshaver
wshaver reviewed May 17, 2013
View reviewed changes
source/includes/parameters-map-reduce.rst
@@ -37,7 +37,8 @@

Optional. Sorts the *input* documents. This option is useful for
optimization. For example, specify the sort key to be the same as
the emit key so that there are fewer reduce operations.
the emit key so that there are fewer reduce operations. The sort
key must be in an existing index for this collection.
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This may be a bug instead of a feature to be documented. Without an index:
errmsg: 'exception: could not create cursor over site.sessions for query : {} sort : { userId: 1 }',
code: 16052,

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://jira.mongodb.org/browse/SERVER-2269 Appears to be expected behavior, should be added to documentation.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Will -
Thank you so much for all your time and effort on this matter and apologies for the lateness of response. We will get to these shortly -- I'm hoping for end of day today.

Regards,

Kay

@kay-kim
Copy link
Contributor

kay-kim commented May 17, 2013

Hi Will --
thank you so much for your contribution and will get right on your pull request. If you could first fill out the Contributor Agreement, that would be greatly appreciated. You can find the agreement at http://www.10gen.com/legal/contributor-agreement.

Regards,

Kay

@kay-kim
Copy link
Contributor

kay-kim commented May 22, 2013

I've merged your changes. Thanks again.

Regards,

Kay

@kay-kim kay-kim closed this May 22, 2013
mongo-cr-bot pushed a commit that referenced this pull request Apr 21, 2022
@jason-price-mongodb
DOCSP-22038-wired-tiger-checkpoint (#998) (#1003)
6ad22d5 
Co-authored-by: jason-price-mongodb <[email protected]>

Co-authored-by: jason-price-mongodb <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wshaver @kay-kim