[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-07 Thread danielyli
Github user danielyli commented on the issue: https://github.com/apache/spark/pull/17793 Thanks all. --- 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

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-07 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/17793 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 feature enabled and wishes so, or i

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 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 does not have this feature e

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76518/ Test PASSed. ---

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-06 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17793 **[Test build #76518 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76518/testReport)** for PR 17793 at commit [`6d27fff`](https://github.com/apache/spark/commit/6

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-06 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17793 **[Test build #76518 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76518/testReport)** for PR 17793 at commit [`6d27fff`](https://github.com/apache/spark/commit/6d

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 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 does not have this feature e

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76516/ Test FAILed. ---

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-05 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17793 **[Test build #76516 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76516/testReport)** for PR 17793 at commit [`3d5d8a6`](https://github.com/apache/spark/commit/3d

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-05 Thread danielyli
Github user danielyli commented on the issue: https://github.com/apache/spark/pull/17793 javaunibuild build results: 1. Doc for `ALS.train` looks fine. 1. No doc for `ALS.OutBlock` is generated; possibly because it's a type def? 1. No doc for `ALS.InBlock` is genera

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-05 Thread danielyli
Github user danielyli commented on the issue: https://github.com/apache/spark/pull/17793 @srowen Great idea. Will do and report back. --- 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 feat

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-04 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/17793 @danielyli I wonder if you can build the docs to make sure that all your comments render as expected? there's a fair bit of formatting going on here and the scaladoc markdown can be surprising. ---

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76430/ Test PASSed. ---

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 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 does not have this feature e

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-03 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17793 **[Test build #76430 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76430/testReport)** for PR 17793 at commit [`c82501a`](https://github.com/apache/spark/commit/c

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-03 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17793 **[Test build #76430 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76430/testReport)** for PR 17793 at commit [`c82501a`](https://github.com/apache/spark/commit/c8

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-03 Thread danielyli
Github user danielyli commented on the issue: https://github.com/apache/spark/pull/17793 All comments have been addressed. --- 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

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-05-02 Thread danielyli
Github user danielyli commented on the issue: https://github.com/apache/spark/pull/17793 Great. Let me finish adding that one change @sethah requested, and I'll update the PR sometime today. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 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 does not have this feature e

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76321/ Test PASSed. ---

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-30 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17793 **[Test build #76321 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76321/testReport)** for PR 17793 at commit [`e5cdba1`](https://github.com/apache/spark/commit/e

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-30 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17793 **[Test build #76321 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76321/testReport)** for PR 17793 at commit [`e5cdba1`](https://github.com/apache/spark/commit/e5

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 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 does not have this feature e

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17793 **[Test build #76292 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76292/testReport)** for PR 17793 at commit [`5a4eb85`](https://github.com/apache/spark/commit/5

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76292/ Test PASSed. ---

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17793 **[Test build #76292 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76292/testReport)** for PR 17793 at commit [`5a4eb85`](https://github.com/apache/spark/commit/5a

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76289/ Test FAILed. ---

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17793 **[Test build #76289 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76289/testReport)** for PR 17793 at commit [`57de83b`](https://github.com/apache/spark/commit/5

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 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 does not have this feature e

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17793 **[Test build #76289 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76289/testReport)** for PR 17793 at commit [`57de83b`](https://github.com/apache/spark/commit/57

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread sethah
Github user sethah commented on the issue: https://github.com/apache/spark/pull/17793 btw "You can build just the Spark scaladoc by running build/sbt unidoc from the SPARK_PROJECT_ROOT directory." [Link](https://github.com/apache/spark/tree/master/docs) --- If your project is set up

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/17793 You have some javadoc errors . See the full log --- 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 feat

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread danielyli
Github user danielyli commented on the issue: https://github.com/apache/spark/pull/17793 How do I fix the “fails to generate documentation” error? --- 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 n

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17793 **[Test build #76264 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76264/testReport)** for PR 17793 at commit [`0a2edf0`](https://github.com/apache/spark/commit/0

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 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 does not have this feature e

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76264/ Test FAILed. ---

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/17793 **[Test build #76264 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76264/testReport)** for PR 17793 at commit [`0a2edf0`](https://github.com/apache/spark/commit/0a

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-28 Thread MLnick
Github user MLnick commented on the issue: https://github.com/apache/spark/pull/17793 ok to test --- 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

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-27 Thread sethah
Github user sethah commented on the issue: https://github.com/apache/spark/pull/17793 +1 for this change. I'll try to take a look sometime, but maybe after the QA period. Also cc @MLnick. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark issue #17793: [SPARK-20484][MLLIB] Add documentation to ALS code

2017-04-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17793 Can one of the admins verify this patch? --- 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 feat