amplab / graphx

Former GraphX development repository. GraphX has been merged into Apache Spark; please submit pull requests there.
https://github.com/apache/spark
Apache License 2.0
360 stars 103 forks source link

GraphLab bug fix & set start vertex #84

Closed amatsukawa closed 10 years ago

amatsukawa commented 10 years ago

Two changes to GraphLab

  1. Fixed a bug where if a vertex has 0 vertices that scatter to it, it stays active forever.
  2. Add ability to set starting vertices using a predicate, with default being all vertices are active.

For 2 above, debated if we should allow the predicate to operate over edges (essentially starting with a scatter phase), but stuck with the simpler one for now.

AmplabJenkins commented 10 years ago

Can one of the admins verify this patch?

amatsukawa commented 10 years ago

Does talking to Jenkins trigger it? That's awesome. :+1: :clap:

ankurdave commented 10 years ago

Yeah, it watches pull requests: https://wiki.jenkins-ci.org/display/JENKINS/GitHub+pull+request+builder+plugin

I'd rather have real GitHub integration like Travis CI does, though :)

ankurdave commented 10 years ago

Oh, apparently that exists! We have to try it out sometime: https://github.com/litl/leeroy

Edit: Just saw that Jenkins has it too. I'm going to try enabling it...

rxin commented 10 years ago

Jenkins, add to whitelist.

AmplabJenkins commented 10 years ago

Merged build triggered.

AmplabJenkins commented 10 years ago

Merged build started.

AmplabJenkins commented 10 years ago

Merged build finished.

AmplabJenkins commented 10 years ago

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