Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8551
---
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 jkbradley commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-162970671
LGTM. Merging with master and branch-1.6
Thank you!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-162822103
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-162822102
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/8551#issuecomment-162822008
**[Test build #47324 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47324/consoleFull)**
for PR 8551 at commit
[`672ce97`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-162815826
**[Test build #47324 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47324/consoleFull)**
for PR 8551 at commit
[`672ce97`](https://gith
Github user hhbyyh commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-162812998
Thanks @jkbradley @BenFradet for the review.
---
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 proje
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-162665829
Those 3 items (thanks @BenFradet ) are the only issues I see. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-162660020
**[Test build #2177 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2177/consoleFull)**
for PR 8551 at commit
[`ee4832d`](https://gi
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/8551#discussion_r46874564
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/LDAExample.scala ---
@@ -192,115 +192,45 @@ object LDAExample {
vocabSize: Int,
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/8551#discussion_r46874402
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/LDAExample.scala ---
@@ -18,9 +18,9 @@
// scalastyle:off println
package org.apa
Github user BenFradet commented on a diff in the pull request:
https://github.com/apache/spark/pull/8551#discussion_r46874064
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/LDAExample.scala ---
@@ -192,115 +192,45 @@ object LDAExample {
vocabSize: Int,
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-162653524
**[Test build #2177 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2177/consoleFull)**
for PR 8551 at commit
[`ee4832d`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-155384291
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-155384289
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/8551#issuecomment-155384164
**[Test build #45515 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45515/consoleFull)**
for PR 8551 at commit
[`ee4832d`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-155378185
**[Test build #45515 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45515/consoleFull)**
for PR 8551 at commit
[`ee4832d`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-155377106
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/8551#issuecomment-155377078
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 feynmanliang commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-155376709
Jenkins retest 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 hhbyyh commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-155264646
unrelated UT timeout failure:
SPARK-11188 Analysis error reporting *** FAILED *** (2 minutes)
[info] ===
[info] CliSuite failure
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-155253327
**[Test build #2026 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2026/consoleFull)**
for PR 8551 at commit
[`ee4832d`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-155240352
**[Test build #2026 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2026/consoleFull)**
for PR 8551 at commit
[`ee4832d`](https://git
Github user hhbyyh commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-147488013
need to trigger the test again?
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-147310376
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/8551#issuecomment-147310378
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-147310348
[Test build #43561 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43561/console)
for PR 8551 at commit
[`ee4832d`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-147305874
[Test build #43561 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43561/consoleFull)
for PR 8551 at commit
[`ee4832d`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-147304573
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/8551#issuecomment-147304561
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 hhbyyh commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-147304609
@feynmanliang @jkbradley Sorry for the late update.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user hhbyyh commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-140443465
@feynmanliang Thanks for the review. I'll update later today.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/8551#discussion_r39460900
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/LDAExample.scala ---
@@ -186,121 +186,52 @@ object LDAExample {
* Load document
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/8551#discussion_r39460837
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/LDAExample.scala ---
@@ -186,121 +186,52 @@ object LDAExample {
* Load document
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/8551#discussion_r39460880
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/LDAExample.scala ---
@@ -186,121 +186,52 @@ object LDAExample {
* Load document
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/8551#discussion_r39460759
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/LDAExample.scala ---
@@ -186,121 +186,52 @@ object LDAExample {
* Load document
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/8551#discussion_r39460809
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/LDAExample.scala ---
@@ -186,121 +186,52 @@ object LDAExample {
* Load document
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-136662537
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-136662532
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/8551#issuecomment-136662449
[Test build #41868 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41868/console)
for PR 8551 at commit
[`84cc1be`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-136654355
[Test build #41868 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41868/consoleFull)
for PR 8551 at commit
[`84cc1be`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8551#issuecomment-136651368
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/8551#issuecomment-136651324
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 hhbyyh opened a pull request:
https://github.com/apache/spark/pull/8551
[SPARK-10393] use ML pipeline in LDA example
jira: https://issues.apache.org/jira/browse/SPARK-10393
Since the logic of the text processing part has been moved to ML
estimators/transformers,
44 matches
Mail list logo