Skip to content

Conversation

@adrian-wang
Copy link
Contributor

We can simple treat cross join as inner join without join conditions.

@SparkQA
Copy link

SparkQA commented Aug 26, 2014

QA tests have started for PR 2124 at commit 8c9b7c5.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Aug 26, 2014

QA tests have finished for PR 2124 at commit 8c9b7c5.

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

@marmbrus
Copy link
Contributor

Thanks! I've merged this into master and 1.1.

@asfgit asfgit closed this in 52fbdc2 Aug 26, 2014
asfgit pushed a commit that referenced this pull request Aug 26, 2014
We can simple treat cross join as inner join without join conditions.

Author: Daoyuan Wang <[email protected]>
Author: adrian-wang <[email protected]>

Closes #2124 from adrian-wang/crossjoin and squashes the following commits:

8c9b7c5 [Daoyuan Wang] add a test
7d47bbb [adrian-wang] add cross join support for hql

(cherry picked from commit 52fbdc2)
Signed-off-by: Michael Armbrust <[email protected]>
@adrian-wang adrian-wang deleted the crossjoin branch August 26, 2014 08:03
kayousterhout pushed a commit to kayousterhout/spark-1 that referenced this pull request Aug 27, 2014
We can simple treat cross join as inner join without join conditions.

Author: Daoyuan Wang <[email protected]>
Author: adrian-wang <[email protected]>

Closes apache#2124 from adrian-wang/crossjoin and squashes the following commits:

8c9b7c5 [Daoyuan Wang] add a test
7d47bbb [adrian-wang] add cross join support for hql
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
We can simple treat cross join as inner join without join conditions.

Author: Daoyuan Wang <[email protected]>
Author: adrian-wang <[email protected]>

Closes apache#2124 from adrian-wang/crossjoin and squashes the following commits:

8c9b7c5 [Daoyuan Wang] add a test
7d47bbb [adrian-wang] add cross join support for hql
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.

3 participants