Skip to content

[SPARK-29107][SQL][TESTS] Port window.sql (Part 1) #25816

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

Closed
DylanGuedes wants to merge 8 commits into from
Closed

[SPARK-29107][SQL][TESTS] Port window.sql (Part 1) #25816

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
df2a00a
adds first part of window split
DylanGuedes Sep 17, 2019
e23bced
add new changes to window_part1
DylanGuedes Sep 29, 2019
3d2c538
update to rel4
DylanGuedes Sep 29, 2019
589ddf6
remove query that didnt pass on thrift
DylanGuedes Sep 29, 2019
2eafd2c
update golden file
DylanGuedes Sep 29, 2019
1062e9c
change queries
DylanGuedes Oct 2, 2019
3a5825c
regen golden file
DylanGuedes Oct 8, 2019
1ac60f1
bump window file version and add new jiras
DylanGuedes Oct 13, 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
Prev Previous commit
bump window file version and add new jiras 
Signed-off-by: DylanGuedes <[email protected]>
  • Loading branch information
@DylanGuedes
DylanGuedes committed Oct 13, 2019
commit 1ac60f1dbd23ae5c76151578e4a49e48be5cc043
6 changes: 3 additions & 3 deletions sql/core/src/test/resources/sql-tests/inputs/pgSQL/window_part1.sql
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
-- Portions Copyright (c) 1996-2019, PostgreSQL Global Development Group
--
-- Window Functions Testing
-- https://github.com/postgres/postgres/blob/REL_12_BETA4/src/test/regress/sql/window.sql#L1-L319
-- https://github.com/postgres/postgres/blob/REL_12_STABLE/src/test/regress/sql/window.sql#L1-L319

CREATE TEMPORARY VIEW tenk2 AS SELECT * FROM tenk1;

Expand Down Expand Up @@ -187,13 +187,13 @@ sum(ten) over (partition by four order by ten range between unbounded preceding
last(ten) over (partition by four order by ten range between unbounded preceding and unbounded following)
FROM (select distinct ten, four from tenk1) ss;

-- Failing on thrift server
-- [SPARK-29451] Some queries with divisions in SQL windows are failling in Thrift
-- SELECT four, ten/4 as two,
-- sum(ten/4) over (partition by four order by ten/4 range between unbounded preceding and current row),
-- last(ten/4) over (partition by four order by ten/4 range between unbounded preceding and current row)
-- FROM (select distinct ten, four from tenk1) ss;
Copy link
Member

@dongjoon-hyun dongjoon-hyun Sep 29, 2019

Choose a reason for hiding this comment

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

Ur, did you regenerate the golden files after this removal?


-- Failing on thrift server
-- [SPARK-29451] Some queries with divisions in SQL windows are failling in Thrift
-- SELECT four, ten/4 as two,
-- sum(ten/4) over (partition by four order by ten/4 rows between unbounded preceding and current row),
-- last(ten/4) over (partition by four order by ten/4 rows between unbounded preceding and current row)
Expand Down