Skip to content

Conversation

@junyangq
Copy link
Contributor

What changes were proposed in this pull request?

This PR adds SparkR vignette to branch 2.0, which works as a friendly guidance going through the functionality provided by SparkR.

How was this patch tested?

R unit test.


## Machine Learning

SparkR supports the following machine learning models and algorithms.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This might need to be revised as not all of these are available in 2.0.0. Can we reframe this to put some version numbers next to each algorithm or something like that ?

cc @felixcheung to see if there are any other ideas

@SparkQA
Copy link

SparkQA commented Sep 14, 2016

Test build #65384 has finished for PR 15100 at commit a4d54ba.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Sep 15, 2016

Test build #65425 has finished for PR 15100 at commit 878c9e0.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

Copy link
Contributor

@shivaram shivaram left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @junyangq - LGTM

@shivaram
Copy link
Contributor

Merging this into branch-2.0

asfgit pushed a commit that referenced this pull request Sep 15, 2016
## What changes were proposed in this pull request?

This PR adds SparkR vignette to branch 2.0, which works as a friendly guidance going through the functionality provided by SparkR.

## How was this patch tested?

R unit test.

Author: junyangq <qianjunyang@gmail.com>
Author: Shivaram Venkataraman <shivaram@cs.berkeley.edu>
Author: Junyang Qian <junyangq@databricks.com>

Closes #15100 from junyangq/SPARKR-vignette-2.0.
@shivaram
Copy link
Contributor

@junyangq We need to manually close this PR as github doesn't pick up commits not merged to master. Can you close this when you get a chance ?

@junyangq
Copy link
Contributor Author

Sure, thanks @shivaram

@junyangq junyangq closed this Sep 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants