Skip to content

Conversation

@kjmrknsn
Copy link
Contributor

@kjmrknsn kjmrknsn commented Nov 30, 2018

What changes were proposed in this pull request?

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

How was this patch tested?

NA

@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.

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
## 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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants