Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-206350081
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/5
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-206350080
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-206349899
**[Test build #55108 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55108/consoleFull)**
for PR 6880 at commit
[`c25eae2`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-206336389
**[Test build #55108 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55108/consoleFull)**
for PR 6880 at commit
[`c25eae2`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-206169860
**[Test build #55098 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55098/consoleFull)**
for PR 6880 at commit
[`23316d4`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-206169896
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/5
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-206169890
Merged build finished. Test FAILed.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-206166387
**[Test build #55098 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55098/consoleFull)**
for PR 6880 at commit
[`23316d4`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-204633940
Merged build finished. Test FAILed.
---
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/6880#issuecomment-204633941
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/5
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-204633939
**[Test build #54751 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54751/consoleFull)**
for PR 6880 at commit
[`b088e46`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-204633686
**[Test build #54751 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/54751/consoleFull)**
for PR 6880 at commit
[`b088e46`](https://gith
Github user FlytxtRnD commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-156399657
@yu-iskw @jkbradley any other review comments, please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user FlytxtRnD commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-153564736
@yu-iskw I didn't get your comment on @Since tags. We will be waiting for
further review comments.
---
If your project is set up for it, you can reply to this email a
Github user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-153409313
@FlytxtRnD thank you for the update. We should add `@since` tags in the
first commit.
Btw, I haven't read the original paper carefully yet. I'll review this PR
in te
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-153337608
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 project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-153337609
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-153337507
**[Test build #44917 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44917/consoleFull)**
for PR 6880 at commit
[`b088e46`](https://git
Github user FlytxtRnD commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-153329563
@yu-iskw PR is updated. Shall I include @since to the methods? Or is it
done after getting merged? Please provide any other suggestions, if any.
---
If your project i
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-153327592
**[Test build #44917 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44917/consoleFull)**
for PR 6880 at commit
[`b088e46`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-153327399
Merged build started.
---
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-153327369
Merged build triggered.
---
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 h
Github user yu-iskw commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r43697828
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,279 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user FlytxtRnD commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r43622644
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,279 @@
+/*
+ * Licensed to the Apache Software Foundat
Github user FlytxtRnD commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-152919710
Thank you @yu-iskw for the review comments.. Will update the PR asap
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user yu-iskw commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r43591908
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,279 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user yu-iskw commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r43591894
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,279 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user yu-iskw commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r43591263
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,279 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user yu-iskw commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r43590992
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,279 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user yu-iskw commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r43590450
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,279 @@
+/*
+ * Licensed to the Apache Software Foundatio
Github user FlytxtRnD commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-149156272
@mengxr Could you please have a look into this?
---
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 FlytxtRnD commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-143643138
@mengxr @jkbradley I have incorporated the suggestions and changes and
updated the PR. Could you please take another look ?
---
If your project is set up for it, you
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-142535150
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 project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-142535152
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-142535047
[Test build #42899 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42899/console)
for PR 6880 at commit
[`e796866`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-142527071
[Test build #42899 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42899/consoleFull)
for PR 6880 at commit
[`e796866`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-142526576
Merged build started.
---
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-142526560
Merged build triggered.
---
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 h
Github user FlytxtRnD commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r39827805
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundat
Github user FlytxtRnD commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r39255547
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundat
Github user FlytxtRnD commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-139170303
@mengxr Thank you for all the suggestions. Will update soon
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38974586
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-138687534
@FlytxtRnD I made another pass. Please follow the code style guide closely:
https://cwiki.apache.org/confluence/display/SPARK/Spark+Code+Style+Guide. I
will make another
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38953908
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/clustering/DpMeansSuite.scala ---
@@ -0,0 +1,84 @@
+/*
+ * Licensed to the Apache Software Founda
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38953865
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/clustering/DpMeansSuite.scala ---
@@ -0,0 +1,84 @@
+/*
+ * Licensed to the Apache Software Founda
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38953796
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/clustering/DpMeansSuite.scala ---
@@ -0,0 +1,84 @@
+/*
+ * Licensed to the Apache Software Founda
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38953485
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/clustering/DpMeansSuite.scala ---
@@ -0,0 +1,84 @@
+/*
+ * Licensed to the Apache Software Founda
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38953312
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeansModel.scala ---
@@ -0,0 +1,67 @@
+/*
+ * Licensed to the Apache Software Founda
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952962
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeansModel.scala ---
@@ -0,0 +1,67 @@
+/*
+ * Licensed to the Apache Software Founda
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952794
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952984
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeansModel.scala ---
@@ -0,0 +1,67 @@
+/*
+ * Licensed to the Apache Software Founda
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952809
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952800
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952815
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeansModel.scala ---
@@ -0,0 +1,67 @@
+/*
+ * Licensed to the Apache Software Founda
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952801
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952480
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952500
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952510
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952454
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952539
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952505
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952496
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952319
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952241
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952093
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952012
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38951952
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/DenseDpMeans.scala ---
@@ -0,0 +1,106 @@
+/*
+ * Licensed to the Apache Software Foun
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952088
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952178
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952167
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952320
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38951973
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/DenseDpMeans.scala ---
@@ -0,0 +1,106 @@
+/*
+ * Licensed to the Apache Software Foun
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952170
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952190
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952091
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38951966
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/DenseDpMeans.scala ---
@@ -0,0 +1,106 @@
+/*
+ * Licensed to the Apache Software Foun
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38951957
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/DenseDpMeans.scala ---
@@ -0,0 +1,106 @@
+/*
+ * Licensed to the Apache Software Foun
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38952017
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,247 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r38951960
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/DenseDpMeans.scala ---
@@ -0,0 +1,106 @@
+/*
+ * Licensed to the Apache Software Foun
Github user FlytxtRnD commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-136956338
@mengxr We have updated the JIRA ticket to include the benchmark results as
well..Could you please take a look and give your suggestions?
---
If your project is set u
Github user FlytxtRnD commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-125838477
Thank you @mengxr . We will take a look into the PR you mentioned.We are
looking forward to have DP-Means in the 1.6 release. Thanks a lot for your kind
support.
--
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-125733942
@FlytxtRnD You might need `build/sbt clean` first. Given the review
bandwidth, we may not be able to make this into 1.5. So I will make another
pass after the 1.5 feature
Github user FlytxtRnD commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-125571416
@jkbradley To generate docs, I installed jekyll. In jekyll build command,
it is showing error.
`[info] Done updating.
[error] (catalyst/compile:compile) Compil
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-123170256
@FlytxtRnD To generate the docs, I've always used jekyll (following the
instructions on that same page). I know that builds more than you want, but
does that at leasd
Github user FlytxtRnD commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-122671172
@mengxr @jkbradley Gentle remainder.
---
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 do
Github user FlytxtRnD commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-119924897
@mengxr Could you please tell me how to generate the API docs? I run
build/sbt unidoc as mentioned in
https://github.com/apache/spark/blob/master/docs/README.md. But
Github user FlytxtRnD commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-119907064
@mengxr I have reduced the PR length so that it would be easier for you to
review. The style issues have been fixed wherever they were observed.
I will change the
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-119486668
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-119486527
[Test build #36766 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36766/console)
for PR 6880 at commit
[`907f4f1`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-119470721
[Test build #36766 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36766/consoleFull)
for PR 6880 at commit
[`907f4f1`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-119469656
Merged build started.
---
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-119469583
Merged build triggered.
---
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 h
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r33736631
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeansModel.scala ---
@@ -0,0 +1,149 @@
+/*
+ * Licensed to the Apache Software Found
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-117854395
@FlytxtRnD I haven't checked the implementation yet. Some high-level
comments:
1. Please follow the code style guide. I saw wrong indentation, extra
spacing, ver
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r33736229
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeansModel.scala ---
@@ -0,0 +1,149 @@
+/*
+ * Licensed to the Apache Software Found
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r33736217
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r33736224
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeans.scala ---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r33736213
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/DenseDpMeans.scala ---
@@ -0,0 +1,109 @@
+/*
+ * Licensed to the Apache Software Foun
Github user FlytxtRnD commented on the pull request:
https://github.com/apache/spark/pull/6880#issuecomment-116645007
@mengxr Could you please say your comments on this PR ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user sujkh85 commented on a diff in the pull request:
https://github.com/apache/spark/pull/6880#discussion_r33227311
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/DpMeansModel.scala ---
@@ -0,0 +1,149 @@
+/*
+ * Licensed to the Apache Software Foun
1 - 100 of 111 matches
Mail list logo