Skip to content
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.

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

Sorting on _id gives incorrect results #1648

Closed
jimmyjones2 opened this issue Oct 11, 2014 · 5 comments
Closed

Sorting on _id gives incorrect results #1648

jimmyjones2 opened this issue Oct 11, 2014 · 5 comments
Assignees
Labels
blocker bug Fixes for quality problems that affect the customer experience PR sent
Milestone

Comments

@jimmyjones2
Copy link
Contributor

#1584 allowed _id to be filtered on, but also enabled sorting, but this doesn't appear to give the expected result. I presume if _id was indexed this would work correctly, so sorting should only be enabled in this case?

@jimmyjones2 jimmyjones2 changed the title Disable sorting on _id, unless indexed Sorting on _id gives incorrect results Oct 11, 2014
@rashidkpc rashidkpc added the bug Fixes for quality problems that affect the customer experience label Oct 14, 2014
@rashidkpc rashidkpc added this to the 4.0.0-BETA3 milestone Oct 14, 2014
@rashidkpc
Copy link
Contributor

Definitely broken, the current work around is to index _id. Marking as bug.

@rashidkpc rashidkpc self-assigned this Nov 12, 2014
@rashidkpc rashidkpc assigned spalger and unassigned rashidkpc Dec 4, 2014
@spalger spalger added the PR sent label Dec 8, 2014
@spalger
Copy link
Contributor

spalger commented Dec 8, 2014

Fixed by #2113

@spalger spalger closed this as completed Dec 8, 2014
@jimmyjones2
Copy link
Contributor Author

I've just tried the latest master and I'm still getting the same behavior. I don't think its actually possible to sort on _id unless it's indexed, I did think about sorting internally on _uid, but this would only order on _id correctly if the type was the same.

@spalger
Copy link
Contributor

spalger commented Dec 12, 2014

Interesting. I'm also on the latest master, don't have _id indexed, and receive no sorting option:

2014-12-04 08_42_32

@jimmyjones2
Copy link
Contributor Author

Sorry @spenceralger clearly hadn't cleared my cache!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocker bug Fixes for quality problems that affect the customer experience PR sent
Projects
None yet
Development

No branches or pull requests

3 participants