[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

2017-05-08 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/17884 merged to master/2.2 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] spark issue #17884: [SparkR][Doc] fix typo in vignettes

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

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

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

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

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

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

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

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

2017-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17884 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

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

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

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

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

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

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

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

2017-05-06 Thread actuaryzhang
Github user actuaryzhang commented on the issue: https://github.com/apache/spark/pull/17884 @felixcheung I ran a quick QA on the vignettes and fixed some additional typos and styles. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

2017-05-06 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/17884 @actuaryzhang thanks - would you have a chance to run a quick QA check on the rest of the vignettes, if you haven't already? --- If your project is set up for it, you can reply to this email

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

2017-05-06 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/17884 This test seems flaky on AppVeyor, not sure why ``` Failed - 1. Error: spark.glm and predict

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

2017-05-06 Thread actuaryzhang
Github user actuaryzhang commented on the issue: https://github.com/apache/spark/pull/17884 @HyukjinKwon Thanks for pointing this out. I will keep this in mind next time. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

2017-05-06 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/17884 I know it is legitimate but It would be worth double checking other typos too. Usually, single typo PR is not encouraged up to my knowledge given reviwing, building and merging costs. --- If

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

2017-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/17884 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

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

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

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

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

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

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

[GitHub] spark issue #17884: [SparkR][Doc] fix typo in vignettes

2017-05-06 Thread actuaryzhang
Github user actuaryzhang commented on the issue: https://github.com/apache/spark/pull/17884 @felixcheung --- 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,