Uh oh!
There was an error while loading. Please reload this page.
[SPARK-18674][SQL][Follow-Up] improve the error message of using join - #16110
[SPARK-18674][SQL][Follow-Up] improve the error message of using join#16110gatorsmile wants to merge 3 commits into
Conversation
gatorsmile
commented
Dec 1, 2016
SparkQA
commented
Dec 2, 2016
Test build #69513 has finished for PR 16110 at commit
|
gatorsmile
commented
Dec 2, 2016
retest this please |
cloud-fan
commented
Dec 2, 2016
shall we put the test in |
rxin
commented
Dec 2, 2016
OK if you are improving this, can you fix the grammar? "USING column |
gatorsmile
commented
Dec 2, 2016
Sure, will do it. Thanks! |
SparkQA
commented
Dec 2, 2016
Test build #69522 has finished for PR 16110 at commit
|
gatorsmile
commented
Dec 2, 2016
Just tried MySQL. The message is not good to me: Will try to improve it by: |
rxin
commented
Dec 2, 2016
That SGTM |
SparkQA
commented
Dec 2, 2016
Test build #69548 has started for PR 16110 at commit |
rxin
commented
Dec 2, 2016
LGTM pending tests. |
gatorsmile
commented
Dec 2, 2016
retest this please |
SparkQA
commented
Dec 2, 2016
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>
cloud-fan
commented
Dec 2, 2016
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> Closesapache#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> Closesapache#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