Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/9974#issuecomment-160788490
I checked the changes after previous PR and they look good. Merged into
master and branch-1.6. Thanks!
---
If your project is set up for it, you can reply to this email
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9974
---
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 feynmanliang commented on the pull request:
https://github.com/apache/spark/pull/9974#issuecomment-159869256
LGTM, though I didn't know what was wrong with the old PR so this should
get a second set of eyes
---
If your project is set up for it, you can reply to this
Github user hhbyyh commented on the pull request:
https://github.com/apache/spark/pull/9974#issuecomment-159897999
@feynmanliang Oh I forgot to add //Examples on and off in JavaLDAExample in
the old PR.
---
If your project is set up for it, you can reply to this email and have your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9974#issuecomment-159630847
**[Test build #46690 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46690/consoleFull)**
for PR 9974 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9974#issuecomment-159636877
**[Test build #46690 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46690/consoleFull)**
for PR 9974 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9974#issuecomment-159637064
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/9974#issuecomment-159637066
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user hhbyyh commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-159604965
@yinxusen Thanks a lot. It seems I cannot reopen the PR. I'll create a new
one.
Yet I keep getting error compiling QueryExecutionListener.scala during
build/sbt
Github user yinxusen commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-159613939
@hhbyyh I am not familiar with `sbt unidoc`, but since it is an example
file, there is no need to generate API docs and check them. You can use
`SKIP_API=1 jekyll
GitHub user hhbyyh opened a pull request:
https://github.com/apache/spark/pull/9974
[SPARK-11689] [ML] Add user guide and example code for LDA under spark.ml
jira: https://issues.apache.org/jira/browse/SPARK-11689
Add simple user guide for LDA under spark.ml and example
Github user hhbyyh commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-159625838
Yes 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 this feature
Github user yinxusen commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-159474280
@hhbyyh The Java example is lack of `// $example on$` and `// $example
off$`.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user yinxusen commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-159474550
@mengxr It's really a problem of testing example code and markdown doc in
the test time. I think Spark QA cannot grasp this kind of errors.
---
If your project is set
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158336112
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/9722#issuecomment-158335989
**[Test build #46408 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46408/consoleFull)**
for PR 9722 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158336115
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9722
---
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 mengxr commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158474752
Merged into master and branch-1.6. 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
Github user feynmanliang commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158329286
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
Github user feynmanliang commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158329258
LGTM
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158332315
**[Test build #46408 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46408/consoleFull)**
for PR 9722 at commit
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158569293
@hhbyyh I reverted the commit because it failed doc build. Could you
re-open this PR and try `SKIP_API=1 jekyll build` on your local. I thought this
is tested in PR
Github user hhbyyh commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158584414
I'll look into it immediately. Sorry for the troubling.
---
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 feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45314139
--- Diff: docs/ml-clustering.md ---
@@ -0,0 +1,25 @@
+---
+layout: global
+title: Clustering - ML
+displayTitle: ML - Clustering
+---
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45382273
--- Diff: docs/ml-guide.md ---
@@ -40,6 +40,7 @@ Also, some algorithms have additional capabilities in the
`spark.ml` API; e.g.,
provide class
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45381957
--- Diff: docs/ml-clustering.md ---
@@ -0,0 +1,31 @@
+---
+layout: global
+title: Clustering - ML
+displayTitle: ML - Clustering
+---
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45382669
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaLDAExample.java ---
@@ -0,0 +1,97 @@
+/*
+ * Licensed to the Apache Software
Github user feynmanliang commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158153601
LGTM after changes
---
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 feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45382120
--- Diff: docs/ml-clustering.md ---
@@ -0,0 +1,31 @@
+---
+layout: global
+title: Clustering - ML
+displayTitle: ML - Clustering
+---
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45428781
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaLDAExample.java ---
@@ -0,0 +1,97 @@
+/*
+ * Licensed to the Apache Software
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158273026
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158273027
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45433223
--- Diff: docs/ml-guide.md ---
@@ -40,6 +40,7 @@ Also, some algorithms have additional capabilities in the
`spark.ml` API; e.g.,
provide class
Github user hhbyyh commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158296762
ERROR: Timeout after 15 minutes
ERROR: Error fetching remote repo 'origin'
hudson.plugins.git.GitException: Failed to fetch from
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45366793
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaLDAExample.java ---
@@ -0,0 +1,97 @@
+/*
+ * Licensed to the Apache Software
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158119154
**[Test build #46336 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46336/consoleFull)**
for PR 9722 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158125993
**[Test build #46336 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46336/consoleFull)**
for PR 9722 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-158126154
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/9722#issuecomment-158126158
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45206456
--- Diff: docs/mllib-guide.md ---
@@ -73,6 +73,7 @@ concepts. It also contains sections on using algorithms
within the Pipelines API
*
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45206436
--- Diff: docs/ml-guide.md ---
@@ -45,6 +45,7 @@ provide class probabilities, and linear models provide
model summaries.
* [Linear methods with
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45207331
--- Diff: docs/ml-clustering.md ---
@@ -0,0 +1,25 @@
+---
+layout: global
+title: Clustering - ML
+displayTitle: ML - Clustering
+---
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45207931
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaLDAExample.java ---
@@ -0,0 +1,97 @@
+/*
+ * Licensed to the Apache Software
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45207539
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaLDAExample.java ---
@@ -0,0 +1,97 @@
+/*
+ * Licensed to the Apache Software
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45206861
--- Diff: docs/ml-clustering.md ---
@@ -0,0 +1,25 @@
+---
+layout: global
+title: Clustering - ML
+displayTitle: ML - Clustering
+---
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45206909
--- Diff: docs/ml-clustering.md ---
@@ -0,0 +1,25 @@
+---
+layout: global
+title: Clustering - ML
+displayTitle: ML - Clustering
+---
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45207955
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/LDAExample.scala ---
@@ -0,0 +1,82 @@
+/*
+ * Licensed to the Apache Software
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45207421
--- Diff: docs/ml-clustering.md ---
@@ -0,0 +1,25 @@
+---
+layout: global
+title: Clustering - ML
+displayTitle: ML - Clustering
+---
Github user feynmanliang commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-157738214
Made a pass
---
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 feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45206678
--- Diff: docs/mllib-guide.md ---
@@ -73,6 +73,7 @@ concepts. It also contains sections on using algorithms
within the Pipelines API
*
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/9722#discussion_r45290684
--- Diff: docs/ml-clustering.md ---
@@ -0,0 +1,25 @@
+---
+layout: global
+title: Clustering - ML
+displayTitle: ML - Clustering
+---
+
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-157518974
cc: @feynmanliang
---
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/9722#issuecomment-157279696
**[Test build #46061 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46061/consoleFull)**
for PR 9722 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-157283389
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/9722#issuecomment-157283293
**[Test build #46061 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46061/consoleFull)**
for PR 9722 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-157283388
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/9722#issuecomment-156814081
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 hhbyyh opened a pull request:
https://github.com/apache/spark/pull/9722
[SPARK-11689] [ML] Add user guide and example code for LDA under spark.ml
jira: https://issues.apache.org/jira/browse/SPARK-11689
Add simple user guide for LDA under spark.ml and example
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-156812972
**[Test build #45951 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45951/consoleFull)**
for PR 9722 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-156814056
**[Test build #45951 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45951/consoleFull)**
for PR 9722 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9722#issuecomment-156814083
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
62 matches
Mail list logo