Skip to content

Conversation

@sethah
Copy link
Contributor

@sethah sethah commented May 18, 2016

What changes were proposed in this pull request?

Correct some typos and incorrectly worded sentences.

How was this patch tested?

Doc changes only.

Note that many of these changes were identified by @whomfire01

@sethah
Copy link
Contributor Author

sethah commented May 18, 2016

cc @holdenk Could you take a look?

<div data-lang="scala" markdown="1">

In the following example, we load rating data from the
In the following example, we load ratings data from the
Copy link
Member

Choose a reason for hiding this comment

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

Although I think "rating data" is actually OK, so is "ratings data". I think the whole change LGTM

@SparkQA
Copy link

SparkQA commented May 18, 2016

Test build #58816 has finished for PR 13180 at commit 564e918.

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

@holdenk
Copy link
Contributor

holdenk commented May 18, 2016

LGTM as well :)

@mengxr
Copy link
Contributor

mengxr commented May 20, 2016

Merged into master and branch-2.0. Thanks!

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

Correct some typos and incorrectly worded sentences.

## How was this patch tested?

Doc changes only.

Note that many of these changes were identified by whomfire01

Author: sethah <[email protected]>

Closes #13180 from sethah/ml_guide_audit.

(cherry picked from commit 5e20350)
Signed-off-by: Xiangrui Meng <[email protected]>
@asfgit asfgit closed this in 5e20350 May 20, 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.

5 participants