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

Support CSVs with headers #1982

Closed
umanwizard opened this issue Feb 12, 2020 · 2 comments · Fixed by #2430
Closed

Support CSVs with headers #1982

umanwizard opened this issue Feb 12, 2020 · 2 comments · Fixed by #2430
Assignees
@sploiselle
Labels
A-dataflow Area: dataflow C-feature Category: new feature or request D-good second issue Difficulty: Good for a newcomer who's warmed up
Milestone
0.2

Comments

@umanwizard
Copy link
Contributor

Column names should be taken automatically from the header
@umanwizard umanwizard added the C-feature Category: new feature or request label Feb 12, 2020
@umanwizard umanwizard self-assigned this Feb 12, 2020
@cuongdo cuongdo added D-good second issue Difficulty: Good for a newcomer who's warmed up A-dataflow Area: dataflow labels Feb 16, 2020
@rjnn rjnn added this to the 0.2 milestone Feb 18, 2020
@rjnn
Copy link
Contributor

rjnn commented Mar 20, 2020

If we specify that headers exist, then we can also use the headers to infer the number of columns.

@cuongdo cuongdo assigned sploiselle and unassigned umanwizard Mar 25, 2020
@cuongdo
Copy link
Contributor

cuongdo commented Mar 25, 2020

Reassigning to @sploiselle to balance issue load

@sploiselle sploiselle mentioned this issue Mar 26, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sploiselle sploiselle

Labels
A-dataflow Area: dataflow C-feature Category: new feature or request D-good second issue Difficulty: Good for a newcomer who's warmed up
Projects
None yet
Milestone
0.2
Development

Successfully merging a pull request may close this issue.

sql: support CSVs with headers sploiselle/materialize
4 participants
@cuongdo @rjnn @umanwizard @sploiselle