-
Notifications
You must be signed in to change notification settings - Fork 28.1k
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.
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
[SPARK-21366][SQL][TEST] Add sql test for window functions #18591
Conversation
Anything we remove from the existing suite |
Test build #79471 has finished for PR 18591 at commit
|
retest this please. |
@gatorsmile AFAIK the |
Test build #79484 has finished for PR 18591 at commit
|
also cc @cloud-fan |
At least these three test cases are not related to Hive compatibility in |
Okay, let me remove them. |
Test build #79498 has finished for PR 18591 at commit
|
Test build #79508 has finished for PR 18591 at commit
|
Test build #79512 has finished for PR 18591 at commit
|
## What changes were proposed in this pull request? Add sql test for window functions, also remove uncecessary test cases in `WindowQuerySuite`. ## How was this patch tested? Added `window.sql` and the corresponding output file. Author: Xingbo Jiang <xingbo.jiang@databricks.com> Closes #18591 from jiangxb1987/window. (cherry picked from commit 66d2168) Signed-off-by: Wenchen Fan <wenchen@databricks.com>
thanks, mering to master/2.2! We may have bug fixes for window functions in 2.2, and this test may be useful. |
## What changes were proposed in this pull request? Add sql test for window functions, also remove uncecessary test cases in `WindowQuerySuite`. ## How was this patch tested? Added `window.sql` and the corresponding output file. Author: Xingbo Jiang <xingbo.jiang@databricks.com> Closes apache#18591 from jiangxb1987/window. (cherry picked from commit 66d2168) Signed-off-by: Wenchen Fan <wenchen@databricks.com>
What changes were proposed in this pull request?
Add sql test for window functions, also remove uncecessary test cases in
WindowQuerySuite
.How was this patch tested?
Added
window.sql
and the corresponding output file.