Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/3610#issuecomment-70894019
Merged into master. Thanks!
---
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 t
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3610#issuecomment-70888427
[Test build #25891 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25891/consoleFull)
for PR 3610 at commit
[`a2ebbd3`](https://gith
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/3610
---
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 enab
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3610#issuecomment-70888441
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3610#issuecomment-70874289
[Test build #25891 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/25891/consoleFull)
for PR 3610 at commit
[`a2ebbd3`](https://githu
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/3610#issuecomment-69433216
@nxwhite-str There are few minor comments left. Do you have time to update
the PR?
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3610#issuecomment-67887528
[Test build #551 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/551/consoleFull)
for PR 3610 at commit
[`f8d5928`](https://githu
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/3610#discussion_r22184615
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/KMeans.scala ---
@@ -43,13 +43,14 @@ class KMeans private (
private var runs: Int,
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3610#issuecomment-67878052
[Test build #551 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/551/consoleFull)
for PR 3610 at commit
[`f8d5928`](https://github
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/3610#issuecomment-67877997
failure in a streaming test...retesting
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3610#issuecomment-67687723
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3610#issuecomment-67687719
[Test build #24654 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24654/consoleFull)
for PR 3610 at commit
[`f8d5928`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3610#issuecomment-67678725
[Test build #24654 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24654/consoleFull)
for PR 3610 at commit
[`f8d5928`](https://githu
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/3610#issuecomment-67678108
ok to test
---
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
enabl
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/3610#issuecomment-67607352
LGTM except minor inline comments.
---
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
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3610#discussion_r22077336
--- Diff: python/pyspark/mllib/tests.py ---
@@ -129,6 +129,23 @@ def test_clustering(self):
self.assertEquals(clusters.predict(data[0]),
clusters.
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3610#discussion_r22077324
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/KMeans.scala ---
@@ -353,6 +360,31 @@ object KMeans {
}
/**
+ * Trai
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3610#discussion_r22077332
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/clustering/KMeansSuite.scala ---
@@ -90,6 +90,27 @@ class KMeansSuite extends FunSuite with
MLlibTestSpa
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3610#discussion_r22077328
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/clustering/KMeansSuite.scala ---
@@ -90,6 +90,27 @@ class KMeansSuite extends FunSuite with
MLlibTestSpa
19 matches
Mail list logo
