GitHub user HyukjinKwon opened a pull request:

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

    [SPARK-21640][SQL][PYTHON][R][FOLLOWUP] Add errorifexists in SparkR and 
other documentations

    ## What changes were proposed in this pull request?
    
    This PR proposes to add `errorifexists` to SparkR API and fix the rest of 
them describing the mode, mainly, in API documentations as well.
    
    This PR also replaces `convertToJSaveMode` to `setWriteMode` so that string 
as is is passed to JVM and executes:
    
    
https://github.com/apache/spark/blob/b034f2565f72aa73c9f0be1e49d148bb4cf05153/sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala#L72-L82
    
    and remove the duplication here:
    
    
https://github.com/apache/spark/blob/3f958a99921d149fb9fdf7ba7e78957afdad1405/sql/core/src/main/scala/org/apache/spark/sql/api/r/SQLUtils.scala#L187-L194
    
    ## How was this patch tested?
    
    Manually checked the built documentation. These were mainly found by `` 
grep -r `error` `` and `grep -r 'error'`.
    
    Also, unit tests added in `test_sparkSQL.R`.

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

    $ git pull https://github.com/HyukjinKwon/spark SPARK-21640-followup

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

    https://github.com/apache/spark/pull/19673.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 #19673
    
----
commit 900ee2908c7bd9613cd5043f72576946c0ecd5e9
Author: hyukjinkwon <[email protected]>
Date:   2017-11-06T03:03:53Z

    Add errorifexists in other places too

----


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to