[GitHub] spark issue #13374: [SPARK-13638][SQL] Add escapeAll option to CSV DataFrame...

2016-07-08 Thread jurriaan
Github user jurriaan commented on the issue: https://github.com/apache/spark/pull/13374 @rxin Sorry for the confusion, fixed it :) --- 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

[GitHub] spark issue #13374: [SPARK-13638][SQL] Add escapeAll option to CSV DataFrame...

2016-07-07 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/13374 Yup... would be great if you can update this. Otherwise 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

[GitHub] spark issue #13374: [SPARK-13638][SQL] Add escapeAll option to CSV DataFrame...

2016-07-07 Thread jurriaan
Github user jurriaan commented on the issue: https://github.com/apache/spark/pull/13374 I thought it should be named in line with the escapeQuotes method, but what it's doing is more like quoting all values then escaping all. So i guess that name could make sense after all --- If

[GitHub] spark issue #13374: [SPARK-13638][SQL] Add escapeAll option to CSV DataFrame...

2016-07-07 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/13374 @jurriaan should this be called quoteAll rather than escapeAll? --- 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