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.

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

Add this warning to the README: #20

Closed
izelnakri opened this issue Jan 27, 2017 · 2 comments
Closed

Add this warning to the README: #20

izelnakri opened this issue Jan 27, 2017 · 2 comments

Comments

@izelnakri
Copy link

http://use-the-index-luke.com/no-offset
@drewolson
Copy link
Owner

drewolson commented Jan 27, 2017 •
edited

I'd prefer to have a dialogue about actually fixing the problem (if there's a good approach).

@drewolson
Copy link
Owner

For example, we could inspect the query, looking for the existence of an order_by clause. If one exists, we could use a smarter approach. I'm all ears.

@vhf vhf mentioned this issue Apr 13, 2017
Closed
@egeersoz egeersoz mentioned this issue May 20, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@drewolson @izelnakri