Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MINOR][DOCS] s/It take/It takes/g #20108

Closed
wants to merge 1 commit into from
Closed

Conversation

jkremser
Copy link
Contributor

What changes were proposed in this pull request?

Fixing three small typos in the docs, in particular:

It take a RDD -> It takes an RDD (twice)

It take an JavaRDD -> It takes a JavaRDD

I didn't create any Jira issue for this minor thing, I hope it's ok.

How was this patch tested?

visually by clicking on 'preview'

@srowen
Copy link
Member

srowen commented Dec 29, 2017

That's OK. It'd be ideal to try to find more typos at once if you can -- a quick spell check can turn up more. No need to get all of them but helps avoid PRs for fixing one at a time.

@SparkQA
Copy link

SparkQA commented Dec 31, 2017

Test build #4032 has finished for PR 20108 at commit daae351.

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

@srowen
Copy link
Member

srowen commented Dec 31, 2017

This time I'll just merge it.

@asfgit asfgit closed this in 5955a2d Dec 31, 2017
@jkremser jkremser deleted the docs-typo branch January 1, 2018 21:19
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