-
Notifications
You must be signed in to change notification settings - Fork 28.8k
[SPARK-18674][SQL][Follow-Up] improve the error message of using join #16110
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
Test build #69513 has finished for PR 16110 at commit
|
retest this please |
shall we put the test in |
OK if you are improving this, can you fix the grammar? "USING column |
Sure, will do it. Thanks! |
Test build #69522 has finished for PR 16110 at commit
|
Just tried MySQL. The message is not good to me:
Will try to improve it by:
|
That SGTM |
Test build #69548 has started for PR 16110 at commit |
LGTM pending tests. |
retest this please |
Test build #69554 has finished for PR 16110 at commit
|
### What changes were proposed in this pull request? Added a test case for using joins with nested fields. ### How was this patch tested? N/A Author: gatorsmile <gatorsmile@gmail.com> Closes #16110 from gatorsmile/followup-18674. (cherry picked from commit 2f8776c) Signed-off-by: Wenchen Fan <wenchen@databricks.com>
thanks, merging to master/2.1! |
### What changes were proposed in this pull request? Added a test case for using joins with nested fields. ### How was this patch tested? N/A Author: gatorsmile <gatorsmile@gmail.com> Closes apache#16110 from gatorsmile/followup-18674.
### What changes were proposed in this pull request? Added a test case for using joins with nested fields. ### How was this patch tested? N/A Author: gatorsmile <gatorsmile@gmail.com> Closes apache#16110 from gatorsmile/followup-18674.
What changes were proposed in this pull request?
Added a test case for using joins with nested fields.
How was this patch tested?
N/A