[GitHub] spark issue #23185: [MINOR][Docs] Fix typos

2018-11-30 Thread kjmrknsn
Github user kjmrknsn commented on the issue:

https://github.com/apache/spark/pull/23185
  
Thanks for checking and merging. I used a spell checker software and 
checked its result.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #23145: [MINOR][Docs][WIP] Fix Typos

2018-11-29 Thread kjmrknsn
Github user kjmrknsn commented on the issue:

https://github.com/apache/spark/pull/23145
  
Thanks for reviewing and merging.
I've just finished checking the whole documentation.
Here is the complete version of this PR : 
https://github.com/apache/spark/pull/23185
Thanks.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request #23185: [MINOR][Docs] Fix typos

2018-11-29 Thread kjmrknsn
GitHub user kjmrknsn opened a pull request:

https://github.com/apache/spark/pull/23185

[MINOR][Docs] Fix typos

## What changes were proposed in this pull request?
Fix Typos.

## How was this patch tested?
NA

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/kjmrknsn/spark docUpdate

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/23185.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #23185


commit 70fc30d1e0eac795c6a230f7255b7e488b1a57cf
Author: Keiji Yoshida 
Date:   2018-11-26T15:29:16Z

[MINOR][Docs] Fix typos




---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #23145: [MINOR][Docs][WIP] Fix Typos

2018-11-27 Thread kjmrknsn
Github user kjmrknsn commented on the issue:

https://github.com/apache/spark/pull/23145
  
Sorry, this PR is still WIP. I'll continue to check the documentation 
later. Thanks.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request #23145: [MINOR][Docs][WIP] Fix Typos

2018-11-27 Thread kjmrknsn
Github user kjmrknsn commented on a diff in the pull request:

https://github.com/apache/spark/pull/23145#discussion_r236909978
  
--- Diff: docs/index.md ---
@@ -67,7 +67,7 @@ Example applications are also provided in Python. For 
example,
 ./bin/spark-submit examples/src/main/python/pi.py 10
 
 Spark also provides an experimental [R API](sparkr.html) since 1.4 (only 
DataFrames APIs included).
--- End diff --

OK, `experimental` has been removed. 
https://github.com/apache/spark/pull/23145/files#diff-1a523bd9fa0dbf998008b37579210e12


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request #23145: [MINOR][Docs] "a R interpreter" -> "an R interpre...

2018-11-26 Thread kjmrknsn
Github user kjmrknsn commented on a diff in the pull request:

https://github.com/apache/spark/pull/23145#discussion_r236549454
  
--- Diff: docs/index.md ---
@@ -67,7 +67,7 @@ Example applications are also provided in Python. For 
example,
 ./bin/spark-submit examples/src/main/python/pi.py 10
 
 Spark also provides an experimental [R API](sparkr.html) since 1.4 (only 
DataFrames APIs included).
--- End diff --

Should we remove the "experimental" adjective?


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark issue #23145: [MINOR][Docs] "a R interpreter" -> "an R interpreter"

2018-11-26 Thread kjmrknsn
Github user kjmrknsn commented on the issue:

https://github.com/apache/spark/pull/23145
  
OK, I'll check the whole documentation later. Thanks.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request #23145: [MINOR][Docs] "a R interpreter" -> "an R interpre...

2018-11-26 Thread kjmrknsn
GitHub user kjmrknsn opened a pull request:

https://github.com/apache/spark/pull/23145

[MINOR][Docs] "a R interpreter" -> "an R interpreter"

## What changes were proposed in this pull request?
This PR changes the phrase of `a R interpreter` to `an R interpreter` on 
`docs/index.md`.

## How was this patch tested?
NA

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/kjmrknsn/spark docUpdate

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/23145.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #23145


commit 9f2f60b3b80b1ae63c9df38cbc3d5bc1010d4123
Author: Keiji Yoshida 
Date:   2018-11-26T15:29:16Z

[MINOR][Docs] "a R interpreter" -> "an R interpreter"




---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org