Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Aug 27, 2016

What changes were proposed in this pull request?

As part of breaking Optimizer.scala apart, this patch moves various join rules into a single file.

How was this patch tested?

This should be covered by existing tests.

@rxin
Copy link
Contributor Author

rxin commented Aug 27, 2016

Hopefully this one can be merged in branch-2.0.

@yhuai
Copy link
Contributor

yhuai commented Aug 27, 2016

lgtm

@rxin
Copy link
Contributor Author

rxin commented Aug 27, 2016

Merging in master/branch-2.0.

asfgit pushed a commit that referenced this pull request Aug 27, 2016
## What changes were proposed in this pull request?
As part of breaking Optimizer.scala apart, this patch moves various join rules into a single file.

## How was this patch tested?
This should be covered by existing tests.

Author: Reynold Xin <[email protected]>

Closes #14846 from rxin/SPARK-17274.

(cherry picked from commit 718b6ba)
Signed-off-by: Reynold Xin <[email protected]>
@asfgit asfgit closed this in 718b6ba Aug 27, 2016
@SparkQA
Copy link

SparkQA commented Aug 27, 2016

Test build #64519 has finished for PR 14846 at commit 1e08294.

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

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