Skip to content

SQL: Allow multiple fields inside GROUP BY #31793

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
Closed
costin opened this issue Jul 4, 2018 · 1 comment
Closed

SQL: Allow multiple fields inside GROUP BY #31793

costin opened this issue Jul 4, 2018 · 1 comment
Assignees
@costin
Labels
:Analytics/SQL SQL querying >enhancement v6.4.0 v7.0.0-beta1

Comments

@costin
Copy link
Member

costin commented Jul 4, 2018

Currently, SQL restricts GROUP BY to only one field, namely for historical reasons. As there are no technical reasons for this, the restriction should be removed.
@costin costin self-assigned this Jul 4, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search-aggs

costin added a commit to costin/elasticsearch that referenced this issue Jul 5, 2018
@costin
SQL: Remove restriction for single column grouping
a460a19 
For historical reasons SQL restricts GROUP BY to only one field.
This commit removes the restriction and improves the test suite with
multi group by tests.

Close elastic#31793
@costin costin mentioned this issue Jul 5, 2018
Merged
costin added a commit that referenced this issue Jul 6, 2018
@costin
SQL: Remove restriction for single column grouping (#31818)
9ffb26a 
For historical reasons SQL restricts GROUP BY to only one field.
This commit removes the restriction and improves the test suite with
multi group by tests.

Close #31793
costin added a commit that referenced this issue Jul 6, 2018
@costin
SQL: Remove restriction for single column grouping (#31818)
1a01724 
For historical reasons SQL restricts GROUP BY to only one field.
This commit removes the restriction and improves the test suite with
multi group by tests.

Close #31793

(cherry picked from commit 9ffb26a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@costin costin

Labels
:Analytics/SQL SQL querying >enhancement v6.4.0 v7.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@costin @colings86 @elasticmachine