Skip to content

[SPARK-2758] UnionRDD's UnionPartition should not reference parent RDDs - #1675

Closed
rxin wants to merge 2 commits into
apache:masterfrom
rxin:unionrdd
Closed

[SPARK-2758] UnionRDD's UnionPartition should not reference parent RDDs#1675
rxin wants to merge 2 commits into
apache:masterfrom
rxin:unionrdd

Conversation

@rxin

@rxinrxin commented Jul 30, 2014

Copy link
Copy Markdown
Contributor

No description provided.

@rxin

rxin commented Jul 30, 2014

Copy link
Copy Markdown
ContributorAuthor

This blocks #1648.

@tdas please take a look at this to make sure this works well with checkpointing.

@mateiz

Copy link
Copy Markdown
Contributor

Jenkins, test this please

@SparkQA

Copy link
Copy Markdown

QA tests have started for PR 1675. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17513/consoleFull

@mateiz

Copy link
Copy Markdown
Contributor

Looks good to me if it passes tests

@tdas

tdas commented Jul 31, 2014

Copy link
Copy Markdown
Contributor

Same as what Matei said.

@SparkQA

Copy link
Copy Markdown

QA results for PR 1675:
- This patch FAILED unit tests.
- This patch merges cleanly
- This patch adds no public classes

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17513/consoleFull

@rxin

rxin commented Jul 31, 2014

Copy link
Copy Markdown
ContributorAuthor

Jenkins, retest this please.

@SparkQA

Copy link
Copy Markdown

QA tests have started for PR 1675. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17524/consoleFull

@SparkQA

Copy link
Copy Markdown

QA results for PR 1675:
- This patch PASSES unit tests.
- This patch merges cleanly
- This patch adds no public classes

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17524/consoleFull

@rxin

rxin commented Jul 31, 2014

Copy link
Copy Markdown
ContributorAuthor

Jenkins, retest this please.

@SparkQA

Copy link
Copy Markdown

QA tests have started for PR 1675. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17545/consoleFull

@SparkQA

Copy link
Copy Markdown

QA results for PR 1675:
- This patch PASSES unit tests.
- This patch merges cleanly
- This patch adds no public classes

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17545/consoleFull

@rxin

rxin commented Jul 31, 2014

Copy link
Copy Markdown
ContributorAuthor

Merging this in master.

@rxin
rxin deleted the unionrdd branch August 13, 2014 08:02
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
Author: Reynold Xin <rxin@apache.org>
Closesapache#1675 from rxin/unionrdd and squashes the following commits:
941d316 [Reynold Xin] Clear RDDs for checkpointing.
c9f05f2 [Reynold Xin] [SPARK-2758] UnionRDD's UnionPartition should not reference parent RDDs
sunchao pushed a commit to sunchao/spark that referenced this pull request Jun 2, 2023
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.

4 participants

@rxin@mateiz@SparkQA@tdas