search

lweides / column-store

0 stars 0 forks source link

Implement API for .csv #18

Closed lweides closed 11 months ago

lweides commented 1 year ago

Implement the API (#9, #10) for .csv files (which are row-stores)

Make sure the implementation is well tested and testable.

Maybe some test cases can be general, and not specific to the .csv implementation.

Eliasrpx commented 1 year ago

Option A: implement custom .csv reader Option B: use known tool

Eliasrpx commented 11 months ago

"Would also be nice to compare this to a very bare-bone .csv reader, which reads the records one at a time. But don't change your implementation now, just as a thought for afterwards" ~https://github.com/lweides/column-store/pull/28#discussion_r1405458223

lweides commented 11 months ago

Can this be closed?

Eliasrpx commented 11 months ago

depends if we want to implement the "barebone csv Reader" later on. I would say we could close it for now and if we decide to add this reader we just re-open this issue