-
Notifications
You must be signed in to change notification settings - Fork 442
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use the same library when Reading and Writing CSV #30
base: master
Are you sure you want to change the base?
Conversation
abendada
commented
Mar 19, 2015
- Added Escape Character
- Use CSVPrinter when writing CSV Files
@abendada Would you add tests for this and make it pass? |
I added a test for escape
Note: quote and escape should have Character as type so nullable. |
@@ -33,7 +33,8 @@ case class CsvRelation protected[spark] ( | |||
location: String, | |||
useHeader: Boolean, | |||
delimiter: Char, | |||
quote: Char, | |||
quote: Character, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use Char
@abendada left a few comments. Would you please also update the README for the new escape character option, and rebase this? Thanks! |