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.

Sign up for GitHub

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

Jump to bottom

[MINOR][Docs] Fix typos #23185

Closed
wants to merge 1 commit into from
Closed

[MINOR][Docs] Fix typos #23185

wants to merge 1 commit into from

Conversation

kjmrknsn
Copy link
Contributor

@kjmrknsn kjmrknsn commented Nov 30, 2018 •
edited

What changes were proposed in this pull request?

Fix Typos.
This PR is the complete version of #23145.

How was this patch tested?

NA

@kjmrknsn kjmrknsn mentioned this pull request Nov 30, 2018
Closed
@HyukjinKwon
Copy link
Member

ok to test

@HyukjinKwon
Copy link
Member

Thanks for skimming the whole doc. cc @srowen.

@SparkQA
Copy link

SparkQA commented Nov 30, 2018

Test build #99497 has finished for PR 23185 at commit 70fc30d.

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

srowen
srowen approved these changes Nov 30, 2018
View reviewed changes
Copy link
Member

@srowen srowen left a comment

Choose a reason for hiding this comment

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

Looks great. Did you find these by just reading the text, or is there a tool that can detect the grammar and agreement issues? thank you either way, that's a great proofread.

@srowen
Copy link
Member

srowen commented Nov 30, 2018

Merged to master

@asfgit asfgit closed this in c3f27b2 Nov 30, 2018
@kjmrknsn kjmrknsn deleted the docUpdate branch November 30, 2018 15:22
@kjmrknsn
Copy link
Contributor Author

Thanks for checking and merging. I used a spell checker software and checked its result.

jackylee-ch pushed a commit to jackylee-ch/spark that referenced this pull request Feb 18, 2019
@kjmrknsn @jackylee-ch
[MINOR][DOCS] Fix typos
dda9c87 
## What changes were proposed in this pull request?
Fix Typos.
This PR is the complete version of apache#23145.

## How was this patch tested?
NA

Closes apache#23185 from kjmrknsn/docUpdate.

Authored-by: Keiji Yoshida <kjmrknsn@gmail.com>
Signed-off-by: Sean Owen <sean.owen@databricks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@srowen srowen srowen approved these changes

@HyukjinKwon HyukjinKwon HyukjinKwon approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@kjmrknsn @HyukjinKwon @SparkQA @srowen