Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10411
---
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 srowen commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-184675125
Merged to master
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-184335470
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/10411#issuecomment-184335467
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/10411#issuecomment-184335307
**[Test build #51318 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51318/consoleFull)**
for PR 10411 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-184320443
**[Test build #51318 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51318/consoleFull)**
for PR 10411 at commit
Github user BenFradet commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-184130751
Great, I'll do that later today.
---
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
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-184130158
@BenFradet yeah I like your last edit. If you're willing to make that
change and the sentence fragment change I'll merge
---
If your project is set up for it, you can
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-183879749
I'm OK merging 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 project does not have this
Github user BenFradet commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52829428
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaALSExample.java ---
@@ -0,0 +1,131 @@
+/*
+ * Licensed to the Apache Software
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52829527
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaALSExample.java ---
@@ -0,0 +1,131 @@
+/*
+ * Licensed to the Apache Software
Github user BenFradet commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52830051
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaALSExample.java ---
@@ -0,0 +1,131 @@
+/*
+ * Licensed to the Apache Software
Github user BenFradet commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52830580
--- Diff: docs/mllib-collaborative-filtering.md ---
@@ -32,16 +32,16 @@ following parameters:
The standard approach to matrix factorization
Github user BenFradet commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52830587
--- Diff: docs/ml-collaborative-filtering.md ---
@@ -0,0 +1,147 @@
+---
+layout: global
+title: Collaborative Filtering - spark.ml
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-183707910
**[Test build #51242 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51242/consoleFull)**
for PR 10411 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-183717003
**[Test build #51242 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51242/consoleFull)**
for PR 10411 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-183717044
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/10411#issuecomment-183717042
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 srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52728479
--- Diff: docs/ml-collaborative-filtering.md ---
@@ -0,0 +1,148 @@
+---
+layout: global
+title: Collaborative Filtering - spark.ml
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52728448
--- Diff: docs/ml-collaborative-filtering.md ---
@@ -0,0 +1,148 @@
+---
+layout: global
+title: Collaborative Filtering - spark.ml
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52728349
--- Diff: docs/ml-collaborative-filtering.md ---
@@ -0,0 +1,148 @@
+---
+layout: global
+title: Collaborative Filtering - spark.ml
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52728386
--- Diff: docs/ml-collaborative-filtering.md ---
@@ -0,0 +1,148 @@
+---
+layout: global
+title: Collaborative Filtering - spark.ml
Github user BenFradet commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52739112
--- Diff: docs/ml-collaborative-filtering.md ---
@@ -0,0 +1,148 @@
+---
+layout: global
+title: Collaborative Filtering - spark.ml
Github user BenFradet commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-183324222
@srowen thanks for the review, will make the necessary changes.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52728138
--- Diff: docs/ml-collaborative-filtering.md ---
@@ -0,0 +1,148 @@
+---
+layout: global
+title: Collaborative Filtering - spark.ml
Github user BenFradet commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52738715
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/ALSExample.scala ---
@@ -0,0 +1,82 @@
+/*
+ * Licensed to the Apache Software
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52727772
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/ALSExample.scala ---
@@ -0,0 +1,82 @@
+/*
+ * Licensed to the Apache Software
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52727882
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaALSExample.java ---
@@ -0,0 +1,131 @@
+/*
+ * Licensed to the Apache Software
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r52727907
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaALSExample.java ---
@@ -0,0 +1,131 @@
+/*
+ * Licensed to the Apache Software
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-183093903
@srowen @coderxiang Do you have time to review 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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-182587163
**[Test build #51051 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51051/consoleFull)**
for PR 10411 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-182601948
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/10411#issuecomment-182601737
**[Test build #51051 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51051/consoleFull)**
for PR 10411 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-182601947
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/10411#issuecomment-182263510
Test FAILed.
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/10411#issuecomment-182263500
**[Test build #51032 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51032/consoleFull)**
for PR 10411 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-182263507
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 BenFradet commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-182253960
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-182262594
**[Test build #51032 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51032/consoleFull)**
for PR 10411 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-182089675
**[Test build #50995 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50995/consoleFull)**
for PR 10411 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-182089686
Test FAILed.
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/10411#issuecomment-182089083
**[Test build #50995 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50995/consoleFull)**
for PR 10411 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-182089684
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 BenFradet commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-174000491
pinging @thunterdb and @jkbradley
---
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/10411#issuecomment-167119997
**[Test build #48308 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48308/consoleFull)**
for PR 10411 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-167126114
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/10411#issuecomment-167126043
**[Test build #48308 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48308/consoleFull)**
for PR 10411 at commit
Github user BenFradet commented on a diff in the pull request:
https://github.com/apache/spark/pull/10411#discussion_r48418593
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaALSExample.java ---
@@ -0,0 +1,131 @@
+/*
+ * Licensed to the Apache Software
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-167126115
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/10411#issuecomment-166845081
**[Test build #48234 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48234/consoleFull)**
for PR 10411 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-166845460
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/10411#issuecomment-166845465
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/10411#issuecomment-166834382
**[Test build #48234 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48234/consoleFull)**
for PR 10411 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-166552068
**[Test build #48176 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48176/consoleFull)**
for PR 10411 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-166557203
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/10411#issuecomment-166557205
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/10411#issuecomment-166560317
**[Test build #48176 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48176/consoleFull)**
for PR 10411 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-166560440
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/10411#issuecomment-166557094
**[Test build #48174 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48174/consoleFull)**
for PR 10411 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-166549963
**[Test build #48174 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48174/consoleFull)**
for PR 10411 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-166560438
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/10411#issuecomment-166430031
**[Test build #48130 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48130/consoleFull)**
for PR 10411 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-166438036
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/10411#issuecomment-166438034
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/10411#issuecomment-166437880
**[Test build #48130 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48130/consoleFull)**
for PR 10411 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-166267186
**[Test build #48109 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48109/consoleFull)**
for PR 10411 at commit
GitHub user BenFradet opened a pull request:
https://github.com/apache/spark/pull/10411
[SPARK-12247] [ML] [DOC] Documentation for spark.ml's ALS and collaborative
filtering in general
This documents the implementation of ALS in `spark.ml` with example code in
scala, java and
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-166274719
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/10411#issuecomment-166274716
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/10411#issuecomment-166274432
**[Test build #48109 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48109/consoleFull)**
for PR 10411 at commit
Github user BenFradet commented on the pull request:
https://github.com/apache/spark/pull/10411#issuecomment-166277975
cc @thunterdb
---
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
71 matches
Mail list logo