[GitHub] spark pull request: [Spark-12655] [GraphX] GraphX does not unpersi...

2016-01-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10713 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is ena

[GitHub] spark pull request: [Spark-12655] [GraphX] GraphX does not unpersi...

2016-01-15 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10713#issuecomment-171944544 Merged to master --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this featu

[GitHub] spark pull request: [Spark-12655] [GraphX] GraphX does not unpersi...

2016-01-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10713#issuecomment-170882902 **[Test build #2373 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2373/consoleFull)** for PR 10713 at commit [`0f78c0e`](https://

[GitHub] spark pull request: [Spark-12655] [GraphX] GraphX does not unpersi...

2016-01-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10713#issuecomment-170878837 **[Test build #2373 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2373/consoleFull)** for PR 10713 at commit [`0f78c0e`](https://g

[GitHub] spark pull request: [Spark-12655] [GraphX] GraphX does not unpersi...

2016-01-12 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10713#issuecomment-170877285 @jasoncl I agree with this change, though I think this ends up returning RDDs that depend on other RDDs which you then unpersist before use. I agree with the comment on

[GitHub] spark pull request: [Spark-12655] [GraphX] GraphX does not unpersi...

2016-01-11 Thread apivovarov
Github user apivovarov commented on a diff in the pull request: https://github.com/apache/spark/pull/10713#discussion_r49402012 --- Diff: graphx/src/test/scala/org/apache/spark/graphx/GraphSuite.scala --- @@ -428,4 +428,20 @@ class GraphSuite extends SparkFunSuite with LocalSparkCo

[GitHub] spark pull request: [Spark-12655] [GraphX] GraphX does not unpersi...

2016-01-11 Thread apivovarov
Github user apivovarov commented on a diff in the pull request: https://github.com/apache/spark/pull/10713#discussion_r49402039 --- Diff: graphx/src/test/scala/org/apache/spark/graphx/GraphSuite.scala --- @@ -428,4 +428,20 @@ class GraphSuite extends SparkFunSuite with LocalSparkCo

[GitHub] spark pull request: [Spark-12655] [GraphX] GraphX does not unpersi...

2016-01-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10713#issuecomment-170741952 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your p

[GitHub] spark pull request: [Spark-12655] [GraphX] GraphX does not unpersi...

2016-01-11 Thread jasoncl
GitHub user jasoncl opened a pull request: https://github.com/apache/spark/pull/10713 [Spark-12655] [GraphX] GraphX does not unpersist RDDs Some VertexRDD and EdgeRDD are created during the intermediate step of g.connectedComponents() but unnecessarily left cached after the method i