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

Optimise orderBy using indexes #56

Closed
s-taylor opened this issue Apr 21, 2016 · 2 comments
Closed

Optimise orderBy using indexes #56

s-taylor opened this issue Apr 21, 2016 · 2 comments
Assignees

Comments

@s-taylor
Copy link
Contributor

Right now orderBy is not using our indexes.
We're using .orderBy('date') in all cases rather than .orderBy({index: 'date'}) when an index exists.
We should optimise our query based on if the field requested to use orderBy is indexed or not (which is now identified in our schema).

https://www.rethinkdb.com/api/javascript/order_by/

@kaievns
Copy link
Contributor

kaievns commented Apr 27, 2016

i think the correct syntax is .orderBy(name, {index: 'blah'})

@s-taylor s-taylor self-assigned this Apr 27, 2016
@nwinch
Copy link
Contributor

nwinch commented Apr 29, 2016

Resolved by #67

@nwinch nwinch closed this as completed Apr 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants