Skip to content

[SQL][minor] rename apply for QueryPlanner #6142

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
cloud-fan wants to merge 2 commits into from
Closed

[SQL][minor] rename apply for QueryPlanner #6142

cloud-fan wants to merge 2 commits into from

Conversation

@cloud-fan
Copy link
Contributor

@cloud-fan cloud-fan commented May 14, 2015

A follow-up of #5624

@AmplabJenkins
Copy link

AmplabJenkins commented May 14, 2015

Merged build triggered.

@AmplabJenkins
Copy link

AmplabJenkins commented May 14, 2015

Merged build started.

@SparkQA
Copy link

SparkQA commented May 14, 2015

Test build #32692 has started for PR 6142 at commit 24c5ffe.

marmbrus
marmbrus reviewed May 14, 2015
View reviewed changes
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/QueryPlanner.scala Outdated
Copy link
Contributor

@marmbrus marmbrus May 14, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably not execute. How about plan?

@AmplabJenkins
Copy link

AmplabJenkins commented May 14, 2015

Merged build triggered.

@AmplabJenkins
Copy link

AmplabJenkins commented May 14, 2015

Merged build started.

@SparkQA
Copy link

SparkQA commented May 14, 2015

Test build #32696 has started for PR 6142 at commit 971a92b.

@rxin
Copy link
Contributor

rxin commented May 14, 2015

LGTM

@SparkQA
Copy link

SparkQA commented May 14, 2015

Test build #32692 has finished for PR 6142 at commit 24c5ffe.

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

@AmplabJenkins
Copy link

AmplabJenkins commented May 14, 2015

Merged build finished. Test PASSed.

@AmplabJenkins
Copy link

AmplabJenkins commented May 14, 2015

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

@SparkQA
Copy link

SparkQA commented May 14, 2015

Test build #32696 has finished for PR 6142 at commit 971a92b.

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

@AmplabJenkins
Copy link

AmplabJenkins commented May 14, 2015

Merged build finished. Test PASSed.

@AmplabJenkins
Copy link

AmplabJenkins commented May 14, 2015

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

@rxin
Copy link
Contributor

rxin commented May 14, 2015

I've merged this - but github sync is down so the ticket is not shown as closed.

asfgit pushed a commit that referenced this pull request May 14, 2015
@cloud-fan @rxin
[SQL][minor] rename apply for QueryPlanner
aa8a0f9 
A follow-up of #5624

Author: Wenchen Fan <[email protected]>

Closes #6142 from cloud-fan/tmp and squashes the following commits:

971a92b [Wenchen Fan] use plan instead of execute
24c5ffe [Wenchen Fan] rename apply

(cherry picked from commit f2cd00b)
Signed-off-by: Reynold Xin <[email protected]>
@asfgit asfgit closed this in f2cd00b May 14, 2015
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 28, 2015
@cloud-fan @jeanlyn
[SQL][minor] rename apply for QueryPlanner
1615a10 
A follow-up of apache#5624

Author: Wenchen Fan <[email protected]>

Closes apache#6142 from cloud-fan/tmp and squashes the following commits:

971a92b [Wenchen Fan] use plan instead of execute
24c5ffe [Wenchen Fan] rename apply
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
@cloud-fan @jeanlyn
[SQL][minor] rename apply for QueryPlanner
53118f6 
A follow-up of apache#5624

Author: Wenchen Fan <[email protected]>

Closes apache#6142 from cloud-fan/tmp and squashes the following commits:

971a92b [Wenchen Fan] use plan instead of execute
24c5ffe [Wenchen Fan] rename apply
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
@cloud-fan @nemccarthy
[SQL][minor] rename apply for QueryPlanner
f8dee0a 
A follow-up of apache#5624

Author: Wenchen Fan <[email protected]>

Closes apache#6142 from cloud-fan/tmp and squashes the following commits:

971a92b [Wenchen Fan] use plan instead of execute
24c5ffe [Wenchen Fan] rename apply
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@cloud-fan @AmplabJenkins @SparkQA @rxin @marmbrus