crunch-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nathan Barry (JIRA)" <>
Subject [jira] [Commented] (CRUNCH-564) Add support for using escape character same as open/close quote character
Date Wed, 30 Sep 2015 17:55:04 GMT


Nathan Barry commented on CRUNCH-564:

There is a constructor that requires no options:
And a constructor that requires all options:

Though in the all option constructor we are not enforcing that the caller actually provided
the values nor does it fall back to a default value if not provided.  We could add fallback
to the default if an option isn't provided in the all option constructor.

> Add support for using escape character same as open/close quote character
> -------------------------------------------------------------------------
>                 Key: CRUNCH-564
>                 URL:
>             Project: Crunch
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Muhammad
>            Assignee: Josh Wills
>            Priority: Trivial
>              Labels: csv, csvparser
> As a user I would like to use CSVInputFormat to handle the CSV files following this RFC
> Many developers use Apache StringEscapeUtils.escapeCsv( ) method to escape their CSVs.
The method escapes the CSV following the RFC4180. 
> The CSVLineReader throws exception in such a case. We can enhance the code to support
the CSVs that use escape same as the quote characters.
> I would appreciate a comment, if someone has knowingly rejected the idea due to some
technical limitation or a problem with allowing escape and quote as same characters. By the
way Apache HAWQ seem to get around this issue somehow and reads such CSVs alright.

This message was sent by Atlassian JIRA

View raw message