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.

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

Implement text/csv content encoding for the response writer #7099

Merged
merged 1 commit into from
Aug 10, 2016

Conversation

jsternberg
Copy link
Contributor

@jsternberg jsternberg commented Jul 29, 2016

CSV doesn't offer a way to separate different sheets from each other and
it doesn't really have a standard format. We add a custom "---"
separator (stolen from the YAML format) to differentiate
commands. The number of columns for each sheet is inferred from the
first returned row in each statement since they should all be the same.

@jwilder
Copy link
Contributor

jwilder commented Jul 29, 2016

What about tab separated values (tsv) instead of csv?

@jsternberg
Copy link
Contributor Author

@jwilder is there a link to that? I'm not aware of that format. What's the different between tsv and csv?

@jwilder
Copy link
Contributor

jwilder commented Jul 29, 2016

@corylanou
Copy link
Contributor

I would offer both TSV and CSV formats. They are similar enough to make it easy to support both.

@corylanou
Copy link
Contributor

:shipit:

CSV doesn't offer a way to separate different sheets from each other and
it doesn't really have a standard format. We separate sheets with a
newline so they can be imported into something like Excel or LibreOffice
more easily.

The number of columns for each sheet is inferred from the first returned
row in each statement since they should all be the same.
@jsternberg jsternberg merged commit ad50888 into master Aug 10, 2016
@jsternberg jsternberg deleted the js-csv-writer branch August 10, 2016 20:33
@jwilder jwilder mentioned this pull request Aug 17, 2016
@timhallinflux timhallinflux added this to the 1.1.0 milestone Dec 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants