[GitHub] spark issue #18844: [SPARK-21640] Add errorifexists as a valid string for Er...

Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18844 Thanks! Merging to master --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] spark issue #18844: [SPARK-21640] Add errorifexists as a valid string for Er...

Github user rxin commented on the issue: https://github.com/apache/spark/pull/18844 Ok makes sense. LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,

[GitHub] spark issue #18844: [SPARK-21640] Add errorifexists as a valid string for Er...

Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18844 Compared with the other mode names, the last one is the only one whose string name do not match the SaveMode enum names. Below is the current one. ```Scala case "overwrite" =>

[GitHub] spark issue #18844: [SPARK-21640] Add errorifexists as a valid string for Er...

Github user rxin commented on the issue: https://github.com/apache/spark/pull/18844 Actually why do we need this? Can't you just add Error? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark issue #18844: [SPARK-21640] Add errorifexists as a valid string for Er...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18844 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] spark issue #18844: [SPARK-21640] Add errorifexists as a valid string for Er...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18844 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80266/ Test PASSed. ---

[GitHub] spark issue #18844: [SPARK-21640] Add errorifexists as a valid string for Er...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18844 **[Test build #80266 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80266/testReport)** for PR 18844 at commit

[GitHub] spark issue #18844: [SPARK-21640] Add errorifexists as a valid string for Er...

Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18844 **[Test build #80266 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80266/testReport)** for PR 18844 at commit

[GitHub] spark issue #18844: [SPARK-21640] Add errorifexists as a valid string for Er...

Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18844 test this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,

[GitHub] spark issue #18844: [SPARK-21640] Add errorifexists as a valid string for Er...

Github user rxin commented on the issue: https://github.com/apache/spark/pull/18844 Jenkins, test this please. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] spark issue #18844: [SPARK-21640] Add errorifexists as a valid string for Er...

Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18844 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this