[GitHub] spark pull request: put 'curRequestSize = 0' after 'logDebug' it

2014-07-17 Thread JerryLead
GitHub user JerryLead opened a pull request: https://github.com/apache/spark/pull/1477 put 'curRequestSize = 0' after 'logDebug' it This is a minor change. We should first logDebug($curRequestSize) and then set it to 0. You can merge this pull request into a Git repository by

[GitHub] spark pull request: [SPARK-2125] Add sort flag and move sort into ...

2014-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1210#issuecomment-49391815 QA results for PR 1210:br- This patch FAILED unit tests.br- This patch merges cleanlybr- This patch adds no public classesbrbrFor more information see test

[GitHub] spark pull request: SPARK-2058: Overriding config from SPARK_HOME ...

2014-07-17 Thread lianhuiwang
Github user lianhuiwang commented on a diff in the pull request: https://github.com/apache/spark/pull/997#discussion_r15095873 --- Diff: bin/compute-classpath.sh --- @@ -30,6 +30,11 @@ FWDIR=$(cd `dirname $0`/..; pwd) # Build up classpath

[GitHub] spark pull request: [SPARK-2570] [SQL] Fix the bug of ClassCastExc...

2014-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1475#issuecomment-49393082 QA results for PR 1475:br- This patch PASSES unit tests.br- This patch merges cleanlybr- This patch adds no public classesbrbrFor more information see test

[GitHub] spark pull request: [SPARK-2571] Correctly report shuffle read met...

2014-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1476#issuecomment-49394559 QA results for PR 1476:br- This patch FAILED unit tests.br- This patch merges cleanlybr- This patch adds no public classesbrbrFor more information see test

[GitHub] spark pull request: [MLlib] SPARK-1536: multiclass classification ...

2014-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/886#issuecomment-49395217 QA results for PR 886:br- This patch PASSES unit tests.br- This patch merges cleanlybr- This patch adds no public classesbrbrFor more information see test

[GitHub] spark pull request: Reservoir sampling implementation.

2014-07-17 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/1478 Reservoir sampling implementation. You can merge this pull request into a Git repository by running: $ git pull https://github.com/rxin/spark reservoirSample Alternatively you can review and

[GitHub] spark pull request: Reservoir sampling implementation.

2014-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1478#issuecomment-49395696 QA tests have started for PR 1478. This patch merges cleanly. brView progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16808/consoleFull ---

[GitHub] spark pull request: [MLlib] SPARK-1536: multiclass classification ...

2014-07-17 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/886#issuecomment-49395750 @mengxr LGTM. @manishamde Thanks for the fixes! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: Reservoir sampling implementation.

2014-07-17 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/1478#discussion_r15096878 --- Diff: core/src/main/scala/org/apache/spark/util/random/SamplingUtils.scala --- @@ -17,9 +17,49 @@ package org.apache.spark.util.random

[GitHub] spark pull request: [MLlib] SPARK-1536: multiclass classification ...

2014-07-17 Thread manishamde
Github user manishamde commented on the pull request: https://github.com/apache/spark/pull/886#issuecomment-49396157 @jkbradley Thanks! I just made another very minor commit to format MimaExcludes.scala. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [MLlib] SPARK-1536: multiclass classification ...

2014-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/886#issuecomment-49396292 QA tests have started for PR 886. This patch merges cleanly. brView progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16809/consoleFull ---

[GitHub] spark pull request: Reservoir sampling implementation.

2014-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1478#issuecomment-49396936 QA tests have started for PR 1478. This patch merges cleanly. brView progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16810/consoleFull ---

[GitHub] spark pull request: [SPARK-2571] Correctly report shuffle read met...

2014-07-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1476#issuecomment-49397551 QA tests have started for PR 1476. This patch merges cleanly. brView progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16811/consoleFull ---

[GitHub] spark pull request: put 'curRequestSize = 0' after 'logDebug' it

2014-07-17 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/1477#issuecomment-49397603 Jenkins, test 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

<    1   2   3