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

Ecto.Query.order_by returning incorrect order #208

Closed
hez opened this issue Jun 6, 2022 · 4 comments
Closed

Ecto.Query.order_by returning incorrect order #208

hez opened this issue Jun 6, 2022 · 4 comments
Labels
question Further information is requested

Comments

@hez
Copy link

hez commented Jun 6, 2022

I've tried to narrow this down to the smallest dataset I could (results very as I remove entries and eventually it does the correct sorting). Running this through the raw Exqlite query! returned the same results as through Ecto.

What I expect to happen:

Using order_by should give the same order as running the raw query in sqlite3 command line.

What actually happens:

Order is not identical.

Smallest sample I could create: https://gist.github.com/hez/41e8ab04a8b0f3b419fad6c66083ad04

@warmwaffles
Copy link
Member

Which version are you running? We recently had some changes around ordering here #205

@hez
Copy link
Author

hez commented Jun 7, 2022

Which version are you running? We recently had some changes around ordering here #205

0.11.1

Just noticed there is an 0.11.2, will check that

@warmwaffles warmwaffles added the question Further information is requested label Jun 7, 2022
@hez
Copy link
Author

hez commented Jun 7, 2022

It does appear that #205 and the release of 0.11.2 fixed it! Thanks!

@hez hez closed this as completed Jun 7, 2022
@warmwaffles
Copy link
Member

Not a problem @hez. Happy hacking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants