-
Notifications
You must be signed in to change notification settings - Fork 28.9k
[SPARK-25084][SQL] distribute by on multiple columns may lead to code… #22067
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
Conversation
|
ok to test. |
|
Test build #94547 has finished for PR 22067 at commit
|
|
ok to test. |
|
@jerryshao Could you help to trigger test build please? |
|
Add unit test with a rand() column in 'distribute by' |
|
Test build #94556 has finished for PR 22067 at commit
|
|
retest this please |
|
Test build #94562 has finished for PR 22067 at commit
|
|
Test build #94568 has finished for PR 22067 at commit
|
|
�Seems #22066 has changed the implementation with a similar approach. I will close this one. |
…gen issue
What changes were proposed in this pull request?
"distribute by" on multiple columns may lead to codegen issue
How was this patch tested?
manual test