Skip to content
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

[SQL] Improve column pruning in the optimizer. #378

Closed
wants to merge 3 commits into from
Closed

[SQL] Improve column pruning in the optimizer. #378

wants to merge 3 commits into from

Conversation

marmbrus
Copy link
Contributor

No description provided.

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@AmplabJenkins
Copy link

Merged build finished. All automated tests passed.

@AmplabJenkins
Copy link

All automated tests passed.
Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14000/

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

rxin
rxin reviewed Apr 10, 2014
View reviewed changes
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala

Project(projectList, Join(prunedChild(left), prunedChild(right), joinType, condition))

case Project(project1, Project(project2, child)) =>
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe project1list and project2list to be more consistent with the rest of the file

@AmplabJenkins
Copy link

Merged build finished. All automated tests passed.

@AmplabJenkins
Copy link

All automated tests passed.
Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14010/

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@AmplabJenkins
Copy link

Merged build finished. All automated tests passed.

@AmplabJenkins
Copy link

All automated tests passed.
Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14013/

@rxin
Copy link
Contributor

rxin commented Apr 10, 2014

merged in master & branch-1.0

@asfgit asfgit closed this in f99401a Apr 10, 2014
asfgit pushed a commit that referenced this pull request Apr 10, 2014
@marmbrus @rxin
[SQL] Improve column pruning in the optimizer.
4843ef0 
Author: Michael Armbrust <[email protected]>

Closes #378 from marmbrus/columnPruning and squashes the following commits:

779da56 [Michael Armbrust] More consistent naming.
1a4e9ea [Michael Armbrust] More comments.
2f4e7b9 [Michael Armbrust] Improve column pruning in the optimizer.

(cherry picked from commit f99401a)
Signed-off-by: Reynold Xin <[email protected]>
@marmbrus marmbrus deleted the columnPruning branch April 23, 2014 22:00
pdeyhim pushed a commit to pdeyhim/spark-1 that referenced this pull request Jun 25, 2014
@marmbrus @pdeyhim
[SQL] Improve column pruning in the optimizer.
2dab922 
Author: Michael Armbrust <[email protected]>

Closes apache#378 from marmbrus/columnPruning and squashes the following commits:

779da56 [Michael Armbrust] More consistent naming.
1a4e9ea [Michael Armbrust] More comments.
2f4e7b9 [Michael Armbrust] Improve column pruning in the optimizer.
tangzhankun pushed a commit to tangzhankun/spark that referenced this pull request Jul 25, 2017
@mccheah @ash211
Retry binding server to random port in the resource staging server te…
3ec9410 
…st. (apache#378)

* Retry binding server to random port in the resource staging server test.

* Break if successful start

* Start server in try block.

* FIx scalastyle

* More rigorous cleanup logic. Increment port numbers.

* Move around more exception logic.

* More exception refactoring.

* Remove whitespace

* Fix test

* Rename variable
erikerlandson pushed a commit to erikerlandson/spark that referenced this pull request Jul 28, 2017
@mccheah @foxish
Retry binding server to random port in the resource staging server te…
7d0fa56 
…st. (apache#378)

* Retry binding server to random port in the resource staging server test.

* Break if successful start

* Start server in try block.

* FIx scalastyle

* More rigorous cleanup logic. Increment port numbers.

* Move around more exception logic.

* More exception refactoring.

* Remove whitespace

* Fix test

* Rename variable
bzhaoopenstack pushed a commit to bzhaoopenstack/spark that referenced this pull request Sep 11, 2019
@liusheng
Merge pull request apache#378 from theopenlab/del-share-network-id
2391786 
Remove OS_SHARE_NETWORK_ID
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.
3 participants
@marmbrus @AmplabJenkins @rxin