Skip to content

SQL: Add PIVOT support #46489

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

Merged
costin merged 10 commits into from
Sep 23, 2019
Merged

SQL: Add PIVOT support #46489

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
84cee40
SQL: Add PIVOT support
costin Aug 20, 2019
a0caa34
Merge branch 'master' into sql/piv
costin Sep 17, 2019
48c6a83
Address feedback
costin Sep 17, 2019
3f356ba
Address feedback
costin Sep 17, 2019
7af8f37
Restrict usage of PIVOT
costin Sep 18, 2019
d28984a
Merge remote-tracking branch 'remotes/upstream/master' into sql/piv
costin Sep 18, 2019
d668751
Prevent usage of InnerAgg for now
costin Sep 18, 2019
854d09c
Remove unneeded file
costin Sep 18, 2019
2606f36
Fix typos
costin Sep 23, 2019
7bbe026
Merge remote-tracking branch 'remotes/upstream/master' into sql/piv
costin Sep 23, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 1 addition & 2 deletions x-pack/plugin/sql/qa/src/main/resources/debug/debug.csv-spec
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,7 @@
//
Copy link
Member Author

@costin costin Sep 9, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file shouldn't be here - I'll remove it on the next commit once the feedback lands to avoid an extra build.


debug
SELECT * FROM (SELECT languages, gender, salary FROM test_emp) PIVOT (AVG(salary) FOR gender IN ('M', 'F'));
//SELECT languages FROM (SELECT languages, gender, salary FROM test_emp) PIVOT (AVG(salary) FOR gender IN ('M', 'F'));
SELECT first_name f, last_name l, dep.from_date d FROM test_emp WHERE dep.dep_name = 'Production' ORDER BY f LIMIT 5;

f:s | l:s | d:ts

Expand Down
16 changes: 1 addition & 15 deletions x-pack/plugin/sql/qa/src/main/resources/pivot.csv-spec
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -203,18 +203,4 @@ null |31070.0
3 |26830.0
4 |24646.0
5 |23353.0
;

innerAggPivot
Copy link
Contributor

@matriv matriv Sep 23, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you remove this? Isn't this the case that is supported?

Copy link
Member Author

@costin costin Sep 23, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#46489 (comment)

Copy link
Contributor

@matriv matriv Sep 23, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thx, I got confused and thought that we'd support this more trivial case.

schema::languages:bt|'F':d
SELECT * FROM (SELECT gender, salary, languages FROM test_emp) PIVOT (SUM_OF_SQUARES(salary) FOR gender IN ('M', 'F'));

languages | 'F'
---------------+---------------
null |31070.0
1 |23537.0
2 |25342.0
3 |26830.0
4 |24646.0
5 |23353.0
;
;