Skip to content
This repository has been archived by the owner on Aug 28, 2021. It is now read-only.

Generic csv importer #29

Closed
aboodman opened this issue Jul 13, 2015 · 5 comments
Closed

Generic csv importer #29

aboodman opened this issue Jul 13, 2015 · 5 comments
Assignees

Comments

@aboodman
Copy link
Contributor

See clients/json_importer as an example.

Since csv is a table, we should import it into List<T>. Not sure what best way to determine T is ... maybe look at some existing csv data, but ideas include:

  • If it's common for the first row to be the headers, have a -first-row-is-header flag on the importer that we use to get field names of T.
  • Take a flag that either references or defines a type to use as T.
@arv arv self-assigned this Jul 13, 2015
@arv
Copy link
Contributor

arv commented Jul 13, 2015

Seems like a good starter...

@aboodman
Copy link
Contributor Author

I'm actually not sure of some good sample data off-hand. It'd be good to find a real example of something useful, not just made-up data.

@aboodman
Copy link
Contributor Author

Note that we can't do the List<T> thing until #31 is fixed. For now this can just import a list of maps.

@arv
Copy link
Contributor

arv commented Jul 13, 2015

For sample data there is https://catalog.data.gov/dataset?res_format=CSV

@aboodman
Copy link
Contributor Author

Great, that works.

On Monday, July 13, 2015, Erik Arvidsson notifications@github.com wrote:

For sample data there is https://catalog.data.gov/dataset?res_format=CSV


Reply to this email directly or view it on GitHub
#29 (comment).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants