Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[OrionHDFSSink] The CSV field separator character may be configurable #495

Closed
frbattid opened this issue Jul 25, 2015 · 2 comments
Closed

[OrionHDFSSink] The CSV field separator character may be configurable #495

frbattid opened this issue Jul 25, 2015 · 2 comments
Assignees
@pcoello25
Labels
should techdebt
Milestone
release/0.12.0

Comments

@frbattid
Copy link
Member

When using csv-row and csv-column, the field separator character for CSV format is hardcoded to ','. This may lead to bad interpretation of the CSV y the saved entitie's value contains ','. Thus, in order to move the responsability to the user, the field separator character could be configurable.
@frbattid frbattid mentioned this issue Jul 25, 2015
Merged
@fgalan
Copy link
Member

fgalan commented Jul 25, 2015

Would Cygnus implement separator character escaping (in this case, the character to scape would depend on the one chosen by the user)?

@frbattid frbattid changed the title The CSV field separator character may be configurable in OrionHDFSSink [OrionHDFSSink] The CSV field separator character may be configurable Sep 15, 2015
@frbattid frbattid added this to the release/0.12.0 milestone Jan 8, 2016
@pcoello25 pcoello25 mentioned this issue Jan 11, 2016
Merged
@pcoello25
Copy link
Contributor

Implemented in #699

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pcoello25 pcoello25

Labels
should techdebt
Projects
None yet
Milestone
release/0.12.0
Development

No branches or pull requests
3 participants
@fgalan @frbattid @pcoello25