Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11844
---
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 user MLnick commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-218388466
Merged to master and branch-2.0. 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-218385461
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-218385459
Merged build finished. Test PASSed.
---
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/11844#issuecomment-218385353
**[Test build #58342 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58342/consoleFull)**
for PR 11844 at commit
Github user MLnick commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-218380671
LGTM pending jenkins
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-218378537
**[Test build #58342 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58342/consoleFull)**
for PR 11844 at commit
Github user zhengruifeng commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62798957
--- Diff: examples/src/main/python/ml/bisecting_k_means_example.py ---
@@ -36,21 +35,20 @@
.getOrCreate()
# $example on$
Github user MLnick commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-218376385
@zhengruifeng just a couple last comments to make these examples consistent
with the KMeans examples. Then I think this is ready.
---
If your project is set up for it,
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62798757
--- Diff: examples/src/main/python/ml/bisecting_k_means_example.py ---
@@ -36,21 +35,20 @@
.getOrCreate()
# $example on$
-
Github user zhengruifeng commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62798712
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaBisectingKMeansExample.java
---
@@ -48,26 +43,19 @@ public static void main(String[]
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62798474
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaBisectingKMeansExample.java
---
@@ -48,26 +43,19 @@ public static void main(String[] args)
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-218350185
Merged build finished. Test PASSed.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-218350186
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-218350146
**[Test build #58314 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58314/consoleFull)**
for PR 11844 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-218345974
**[Test build #58314 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58314/consoleFull)**
for PR 11844 at commit
Github user zhengruifeng commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-218345784
@MLnick Thanks. Updated
---
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 zhengruifeng commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62783082
--- Diff: docs/ml-clustering.md ---
@@ -104,4 +104,48 @@ Refer to the [Java API
docs](api/java/org/apache/spark/ml/clustering/LDA.html) f
{%
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62716684
--- Diff: docs/ml-clustering.md ---
@@ -104,4 +104,48 @@ Refer to the [Java API
docs](api/java/org/apache/spark/ml/clustering/LDA.html) f
{%
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62714781
--- Diff: docs/ml-clustering.md ---
@@ -104,4 +104,48 @@ Refer to the [Java API
docs](api/java/org/apache/spark/ml/clustering/LDA.html) f
{%
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62713129
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/BisectingKMeansExample.scala
---
@@ -0,0 +1,67 @@
+/*
+ * Licensed to the Apache
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62712877
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaBisectingKMeansExample.java
---
@@ -17,23 +17,14 @@
package
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62712615
--- Diff: examples/src/main/python/ml/bisecting_k_means_example.py ---
@@ -19,9 +19,6 @@
# $example on$
from pyspark.ml.clustering import
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62711985
--- Diff: docs/ml-clustering.md ---
@@ -104,4 +104,48 @@ Refer to the [Java API
docs](api/java/org/apache/spark/ml/clustering/LDA.html) f
{%
Github user zhengruifeng commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-218015365
@wangmiao1981 Once this PR is merged, you can directly load the datafile in
your PR.
---
If your project is set up for it, you can reply to this email and have
Github user zhengruifeng commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62587969
--- Diff: docs/ml-clustering.md ---
@@ -104,4 +104,48 @@ Refer to the [Java API
docs](api/java/org/apache/spark/ml/clustering/LDA.html) f
{%
Github user wangmiao1981 commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-217954275
@zhengruifeng Can you make it sharing with GMM? Once your PR is merged, I
can change mine to use your data. Thanks!
---
If your project is set up for it, you
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62518857
--- Diff: docs/ml-clustering.md ---
@@ -104,4 +104,48 @@ Refer to the [Java API
docs](api/java/org/apache/spark/ml/clustering/LDA.html) f
{%
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-217869533
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-217869531
Merged build finished. Test PASSed.
---
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/11844#issuecomment-217869383
**[Test build #58144 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58144/consoleFull)**
for PR 11844 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-217866822
Merged build finished. Test PASSed.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-217866824
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-217866690
**[Test build #58143 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58143/consoleFull)**
for PR 11844 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-217860158
**[Test build #58144 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58144/consoleFull)**
for PR 11844 at commit
Github user zhengruifeng commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-217860041
@MLnick sorry to involve other peoples' commits into this. I had to
recreate this pr.
I have updated it according to your comments. Thanks
---
If your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-217857904
**[Test build #58143 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58143/consoleFull)**
for PR 11844 at commit
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62491305
--- Diff: docs/ml-clustering.md ---
@@ -104,4 +104,38 @@ Refer to the [Java API
docs](api/java/org/apache/spark/ml/clustering/LDA.html) f
{%
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62490261
--- Diff: docs/ml-clustering.md ---
@@ -104,4 +104,38 @@ Refer to the [Java API
docs](api/java/org/apache/spark/ml/clustering/LDA.html) f
{%
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62489793
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/BisectingKMeansExample.scala
---
@@ -0,0 +1,67 @@
+/*
+ * Licensed to the Apache
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62489736
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/BisectingKMeansExample.scala
---
@@ -0,0 +1,67 @@
+/*
+ * Licensed to the Apache
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11844#discussion_r62489675
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/BisectingKMeansExample.scala
---
@@ -0,0 +1,67 @@
+/*
+ * Licensed to the Apache
Github user MLnick commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-217850151
Thanks @viirya !
---
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 user zhengruifeng commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-217797280
@MLnick The results are OK now
---
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 viirya commented on the pull request:
https://github.com/apache/spark/pull/11844#issuecomment-217705549
@MLnick I opened a PR #12986 for that.
---
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
45 matches
Mail list logo