Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1407#issuecomment-49130099
QA results for PR 1407:br- This patch FAILED unit tests.br- This patch
merges cleanlybr- This patch adds no public classesbrbrFor more
information see test
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/1407#discussion_r14925960
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/LocalKMeans.scala ---
@@ -59,6 +59,11 @@ private[mllib] object LocalKMeans extends Logging {
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/1407#discussion_r14926024
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/clustering/KMeansSuite.scala ---
@@ -61,6 +61,30 @@ class KMeansSuite extends FunSuite with
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/1407#discussion_r14926012
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/clustering/KMeansSuite.scala ---
@@ -61,6 +61,30 @@ class KMeansSuite extends FunSuite with
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/1407#issuecomment-49012803
@jkbradley The fix looks good to me except some minor style issues. Thanks
for fixing it! Btw, please add `[MLLIB]` to the title so this is easy to find.
---
If your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1407#issuecomment-49098158
QA results for PR 1407:br- This patch PASSES unit tests.br- This patch
merges cleanlybr- This patch adds no public classesbrbrFor more
information see test
GitHub user jkbradley opened a pull request:
https://github.com/apache/spark/pull/1407
SPARK-1215: Clustering: Index out of bounds error
Bug fix for JIRA SPARK 1215: Clustering: Index out of bounds error
https://issues.apache.org/jira/browse/SPARK-1215
Solution:
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/1407#issuecomment-48934486
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
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/1407#issuecomment-48948316
Jenkins, add to whitelist and 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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1407#issuecomment-48949060
QA tests have started for PR 1407. This patch merges cleanly. brView
progress:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16633/consoleFull
---
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1407#issuecomment-48960591
QA results for PR 1407:br- This patch PASSES unit tests.br- This patch
merges cleanlybr- This patch adds no public classesbrbrFor more
information see test
11 matches
Mail list logo