Skip to content

[SPARK-12431][GraphX] Add local checkpointing to GraphX.#13379

Closed
adeandrade wants to merge 2 commits intoapache:branch-1.6from
adeandrade:graphx-local-checkpoint
Closed

[SPARK-12431][GraphX] Add local checkpointing to GraphX.#13379
adeandrade wants to merge 2 commits intoapache:branch-1.6from
adeandrade:graphx-local-checkpoint

Conversation

@adeandrade
Copy link

What changes were proposed in this pull request?

Local checkpointing was added to RDD to speed up iterative spark jobs, but this capability hasn't been added to GraphX.

How was this patch tested?

Manually tested.

@andrewor14
Copy link
Contributor

ok to test

@andrewor14
Copy link
Contributor

andrewor14 commented May 31, 2016

@jkbradley what do you think? Should we just use local checkpointing internally through some flag or something?

@SparkQA
Copy link

SparkQA commented May 31, 2016

Test build #59682 has finished for PR 13379 at commit 159f9d7.

  • This patch fails MiMa tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented May 31, 2016

Test build #59696 has finished for PR 13379 at commit 75fea68.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@adeandrade
Copy link
Author

After fixing the MiMa issue the tests for the graphx module pass. The error in test build #59696 seems unrelated.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@github-actions
Copy link

We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag!

@github-actions github-actions bot added the Stale label Jan 18, 2020
@github-actions github-actions bot closed this Jan 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants