[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-85363752 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/5125 --- 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

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-85363731 [Test build #29054 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29054/consoleFull) for PR 5125 at commit

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5125#discussion_r27003271 --- Diff: core/src/main/scala/org/apache/spark/rdd/JdbcRDD.scala --- @@ -70,7 +71,8 @@ class JdbcRDD[T: ClassTag]( }).toArray } -

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-85338836 [Test build #29053 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29053/consoleFull) for PR 5125 at commit

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-85341286 [Test build #29054 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29054/consoleFull) for PR 5125 at commit

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-85338913 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-85338909 [Test build #29053 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29053/consoleFull) for PR 5125 at commit

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/5125#discussion_r26968907 --- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala --- @@ -986,7 +986,7 @@ class SparkContext(config: SparkConf) extends Logging with

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/5125#discussion_r26966209 --- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala --- @@ -986,7 +986,7 @@ class SparkContext(config: SparkConf) extends Logging with

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5125#discussion_r26966555 --- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala --- @@ -986,7 +986,7 @@ class SparkContext(config: SparkConf) extends Logging with

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5125#discussion_r26966597 --- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala --- @@ -986,7 +986,7 @@ class SparkContext(config: SparkConf) extends Logging with

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/5125#discussion_r26967311 --- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala --- @@ -986,7 +986,7 @@ class SparkContext(config: SparkConf) extends Logging with

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/5125#discussion_r26995821 --- Diff: core/src/main/scala/org/apache/spark/deploy/worker/DriverRunner.scala --- @@ -169,8 +177,8 @@ private[deploy] class DriverRunner(

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/5125#discussion_r26995786 --- Diff: core/src/main/scala/org/apache/spark/deploy/worker/DriverRunner.scala --- @@ -33,6 +33,8 @@ import org.apache.spark.deploy.master.DriverState

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/5125#discussion_r26996110 --- Diff: core/src/main/scala/org/apache/spark/rdd/JdbcRDD.scala --- @@ -70,7 +71,8 @@ class JdbcRDD[T: ClassTag]( }).toArray } -

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/5125#discussion_r26995951 --- Diff: core/src/main/scala/org/apache/spark/metrics/MetricsSystem.scala --- @@ -20,6 +20,8 @@ package org.apache.spark.metrics import

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-23 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-85271574 LGTM overall. --- 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

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-84574953 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/5125 [SPARK-6428] Added explicit types for all public methods in core. You can merge this pull request into a Git repository by running: $ git pull https://github.com/rxin/spark core-explicit-type

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-84546495 [Test build #28957 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28957/consoleFull) for PR 5125 at commit

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-84574940 **[Test build #28957 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28957/consoleFull)** for PR 5125 at commit

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-84677924 [Test build #28968 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28968/consoleFull) for PR 5125 at commit

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-84696159 [Test build #28969 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28969/consoleFull) for PR 5125 at commit

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-84696185 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-84681097 [Test build #28969 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28969/consoleFull) for PR 5125 at commit

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-84676626 Jenkins, retest this please. --- 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

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-84677028 [Test build #28968 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28968/consoleFull) for PR 5125 at commit

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-84677937 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-84680821 Jenkins, retest this please. --- 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

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-84780198 [Test build #28977 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28977/consoleFull) for PR 5125 at commit

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-84810862 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6428] Added explicit types for all publ...

2015-03-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5125#issuecomment-84810844 [Test build #28977 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28977/consoleFull) for PR 5125 at commit