Skip to content

[SPARK-17270][SQL] Move object optimization rules into its own file - #14839

Closed
rxin wants to merge 1 commit into
apache:masterfrom
rxin:SPARK-17270
Closed

[SPARK-17270][SQL] Move object optimization rules into its own file#14839
rxin wants to merge 1 commit into
apache:masterfrom
rxin:SPARK-17270

Conversation

@rxin

@rxinrxin commented Aug 27, 2016

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

As part of breaking Optimizer.scala apart, this patch moves various Dataset object optimization rules into a single file. I'm submitting separate pull requests so we can more easily merge this in branch-2.0 to simplify optimizer backports.

How was this patch tested?

This should be covered by existing tests.

@rxin

rxin commented Aug 27, 2016

Copy link
Copy Markdown
ContributorAuthor

cc @cloud-fan

@SparkQA

Copy link
Copy Markdown

Test build #64510 has finished for PR 14839 at commit 39d8a9e.

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

@yhuai

Copy link
Copy Markdown
Contributor

LGTM. Merging to master and branch 2.0.

@yhuai

Copy link
Copy Markdown
Contributor

hmm. There are conflicts in branch 2.0.

@yhuai

Copy link
Copy Markdown
Contributor

Seems the conflicts are caused by d063898#diff-a636a87d8843eeccca90140be91d4faf.

@rxin

rxin commented Aug 27, 2016

Copy link
Copy Markdown
ContributorAuthor

OK I will submit a separate one.

asfgit pushed a commit that referenced this pull request Aug 27, 2016
…branch-2.0)
## What changes were proposed in this pull request?
As part of breaking Optimizer.scala apart, this patch moves various Dataset object optimization rules into a single file. I'm submitting separate pull requests so we can more easily merge this in branch-2.0 to simplify optimizer backports.
This is #14839 but for branch-2.0.
## How was this patch tested?
This should be covered by existing tests.
Author: Reynold Xin <rxin@databricks.com>
Closes#14843 from rxin/SPARK-17270-branch-2.0.
Sign up for freeto 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

@rxin@SparkQA@yhuai