-
Notifications
You must be signed in to change notification settings - Fork 28.5k
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-27463][PYTHON][FOLLOW-UP] Run the tests of Cogrouped pandas UDF #25890
Conversation
Okay, seems fortunately it's passing:
|
This reverts commit 7d9835d.
I am going to merge this as soon as the tests pass to prevent further unexpected regressions in cogroup pandas UDF. cc @BryanCutler and @ueshin |
Test build #111151 has finished for PR 25890 at commit
|
Test build #111150 has finished for PR 25890 at commit
|
Test build #111152 has finished for PR 25890 at commit
|
Merged to master. |
+1, late LGTM. |
late LGTM. Thanks! |
Thanks for catching this @HyukjinKwon , LGTM |
What changes were proposed in this pull request?
This is a followup for #24981
Seems we mistakenly didn't added
test_pandas_udf_cogrouped_map
intomodules.py
. So we don't have official test results against that PR.If tests fail, we should revert that PR.
Why are the changes needed?
Relevant tests should be ran.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Jenkins tests.