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

sql: add support for ORDER BY #2096

Closed
petermattis opened this issue Aug 13, 2015 · 0 comments
Closed

sql: add support for ORDER BY #2096

petermattis opened this issue Aug 13, 2015 · 0 comments
Milestone
Beta

Comments

@petermattis
Copy link
Collaborator

The initial implementation should just be a new type of planNode which buffers the results and sorts them on the desired columns. Follow-on optimization would be to incorporate ORDER BY requirements into index selection to eliminate the explicit sort when possible.
@petermattis petermattis added this to the v0.1 (Beta) milestone Aug 14, 2015
@tbg tbg added the SQL label Aug 15, 2015
@jess-edwards jess-edwards mentioned this issue Aug 17, 2015
Closed
78 tasks
petermattis added a commit that referenced this issue Aug 19, 2015
@petermattis
Add semantic actions for ORDER BY.
1b09030 
See #2096.
@petermattis petermattis mentioned this issue Aug 19, 2015
Merged
@petermattis petermattis mentioned this issue Aug 19, 2015
Merged
petermattis added a commit that referenced this issue Aug 19, 2015
@petermattis
Add support for ORDER BY.
fab23e0 
Sorting is currently performed in memory. Ordering is not taken into
consideration during index selection, but sorting is not performed if
the output is already in the correct order.

Fixes #2096.
@blathers-crl blathers-crl bot mentioned this issue Jun 21, 2023
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
Beta
Development

No branches or pull requests
2 participants
@tbg @petermattis