[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-30 Thread ep1804
Github user ep1804 commented on the issue: https://github.com/apache/spark/pull/20004 OK. I'll deal with it that after the new year break : ) --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-28 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/20004 Thanks! Merged to master. Could you submit a follow-up PR to address my above comment? --- - To unsubscribe, e-mail:

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-28 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/20004 LGTM --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-26 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/20004 retest this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85393/ Test FAILed. ---

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-25 Thread ep1804
Github user ep1804 commented on the issue: https://github.com/apache/spark/pull/20004 Revision followed: - comment on the default values. - applying charToEscapeQuoteEscaping using Option type. --- - To

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-22 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/20004 Thank you for your investigation! Basically, because the option `escape` is set to `\`, the default value of `charToEscapeQuoteEscaping` is actually `\` in effect. Could you

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85097/ Test FAILed. ---

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-18 Thread ep1804
Github user ep1804 commented on the issue: https://github.com/apache/spark/pull/20004 `CSVOptions` is changed NOT to set `charToEscapeQuoteEscaping` to `\u` by default, to allow the uniVocity parser to use `escape` as `charToEscapeQuoteEscaping` character. ---

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-18 Thread ep1804
Github user ep1804 commented on the issue: https://github.com/apache/spark/pull/20004 When `charToEscapeQuoteEscaping` is not set and `quote` and `escape` are different, uniVocity parser uses `escape` character as `charToEscapeQuoteEscaping` by default. This is why the test passes.

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85096/ Test FAILed. ---

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-18 Thread ep1804
Github user ep1804 commented on the issue: https://github.com/apache/spark/pull/20004 Option name changed to `charToEscapeQuoteEscaping`. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-18 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/20004 Let us use `charToEscapeQuoteEscaping ` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85038/ Test FAILed. ---

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85037/ Test FAILed. ---

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85036/ Test FAILed. ---

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

2017-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark-22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark 22818][SQL] csv escape of quote escape

2017-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85023/ Test FAILed. ---

[GitHub] spark issue #20004: [Spark 22818][SQL] csv escape of quote escape

2017-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20004: [Spark 22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark 22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark 22818][SQL] csv escape of quote escape

2017-12-17 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/20004 retest this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #20004: [Spark 22818][SQL] csv escape of quote escape

2017-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20004: [Spark 22818][SQL] csv escape of quote escape

2017-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85021/ Test FAILed. ---

[GitHub] spark issue #20004: [Spark 22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark 22818][SQL] csv escape of quote escape

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

[GitHub] spark issue #20004: [Spark 22818][SQL] csv escape of quote escape

2017-12-17 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/20004 FYI, we are suffering from R test failures. Probably, it will fail on it currently. Let's fix up the title to `[SPARK-22818][SQL] ...`. ---

[GitHub] spark issue #20004: [Spark 22818][SQL] csv escape of quote escape

2017-12-17 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/20004 ok to test --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #20004: [Spark 22818][SQL] csv escape of quote escape

2017-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20004 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20004: [Spark 22818][SQL] csv escape of quote escape

2017-12-17 Thread ep1804
Github user ep1804 commented on the issue: https://github.com/apache/spark/pull/20004 This is the original thread: https://github.com/apache/spark/pull/17177 --- - To unsubscribe, e-mail:

[GitHub] spark issue #20004: [Spark 22818][SQL] csv escape of quote escape

2017-12-17 Thread ep1804
Github user ep1804 commented on the issue: https://github.com/apache/spark/pull/20004 @HyukjinKwon This issue is re-open. Please check this. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org