Skip to content

Conversation

@debuggingfuture
Copy link
Contributor

should sync up with here?

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@rxin
Copy link
Contributor

rxin commented May 6, 2015

Jenkins, test this please.

@rxin
Copy link
Contributor

rxin commented May 6, 2015

Absolutely. Thanks for catching that.

@rxin
Copy link
Contributor

rxin commented May 6, 2015

Actually while you at it, do you mind throwing an AnalysisException in that apply method for a join method that cannot be matched? Right now it will throw a really confusing exception.

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@SparkQA
Copy link

SparkQA commented May 6, 2015

Test build #31994 has started for PR 5944 at commit fc0e454.

@SparkQA
Copy link

SparkQA commented May 6, 2015

Test build #31994 has finished for PR 5944 at commit fc0e454.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

Merged build finished. Test PASSed.

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31994/
Test PASSed.

@rxin
Copy link
Contributor

rxin commented May 11, 2015

Merging this in so we can have it for 1.4.

asfgit pushed a commit that referenced this pull request May 11, 2015
should sync up with here?
https://github.com/apache/spark/blob/119f45d61d7b48d376cca05e1b4f0c7fcf65bfa8/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/joinTypes.scala#L26

Author: LCY Vincent <[email protected]>

Closes #5944 from vincentlaucy/master and squashes the following commits:

fc0e454 [LCY Vincent] Update DataFrame.scala

(cherry picked from commit a8ea096)
Signed-off-by: Reynold Xin <[email protected]>
@asfgit asfgit closed this in a8ea096 May 11, 2015
@rxin
Copy link
Contributor

rxin commented May 11, 2015

I submitted a patch for that: #6064

asfgit pushed a commit that referenced this pull request May 12, 2015
As a follow-up to #5944

Author: Reynold Xin <[email protected]>

Closes #6064 from rxin/jointype-better-error and squashes the following commits:

7629bf7 [Reynold Xin] [SQL] Show better error messages for incorrect join types in DataFrames.

(cherry picked from commit 4f4dbb0)
Signed-off-by: Reynold Xin <[email protected]>
asfgit pushed a commit that referenced this pull request May 12, 2015
As a follow-up to #5944

Author: Reynold Xin <[email protected]>

Closes #6064 from rxin/jointype-better-error and squashes the following commits:

7629bf7 [Reynold Xin] [SQL] Show better error messages for incorrect join types in DataFrames.
asfgit pushed a commit that referenced this pull request May 12, 2015
As a follow-up to #5944

Author: Reynold Xin <[email protected]>

Closes #6064 from rxin/jointype-better-error and squashes the following commits:

7629bf7 [Reynold Xin] [SQL] Show better error messages for incorrect join types in DataFrames.

(cherry picked from commit 4f4dbb0)
Signed-off-by: Reynold Xin <[email protected]>
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 28, 2015
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 28, 2015
As a follow-up to apache#5944

Author: Reynold Xin <[email protected]>

Closes apache#6064 from rxin/jointype-better-error and squashes the following commits:

7629bf7 [Reynold Xin] [SQL] Show better error messages for incorrect join types in DataFrames.
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
As a follow-up to apache#5944

Author: Reynold Xin <[email protected]>

Closes apache#6064 from rxin/jointype-better-error and squashes the following commits:

7629bf7 [Reynold Xin] [SQL] Show better error messages for incorrect join types in DataFrames.
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
As a follow-up to apache#5944

Author: Reynold Xin <[email protected]>

Closes apache#6064 from rxin/jointype-better-error and squashes the following commits:

7629bf7 [Reynold Xin] [SQL] Show better error messages for incorrect join types in DataFrames.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants