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

Improvements bundle #26

Open
kbilsted opened this issue Feb 1, 2015 · 3 comments
Open

Improvements bundle #26

kbilsted opened this issue Feb 1, 2015 · 3 comments

Comments

@kbilsted
Copy link
Member

kbilsted commented Feb 1, 2015

On http://csveed.org/comparison-matrix.html all the cells with a "no" for the supercsv column is a potential enhancement idea.

  • custom escape
  • deal with leading/trailing whitespace (i thought we had support for that)
  • get by column name (i thought this would be the mapreader)
  • read all lines
  • skip comment lines ((i thought we had support for that)
  • Split cell to multiple properties (well that would be custom processort)
  • Join multiple cells into one property ( @jamesbassett already wrote a processor example for doing this)
  • bean in original context
  • single facade

I think we may need to ask the author of the comparison to revise his sheet due to many misconceptions...

@jamesbassett
Copy link
Contributor

You might also be interested in https://github.com/uniVocity/csv-parsers-comparison

@kbilsted
Copy link
Member Author

kbilsted commented Feb 1, 2015

  1. Are any of these ideas worth making issues for?
  2. should we get the comparison up to speed with all out features?

@ZioberMichal
Copy link
Collaborator

I have created new issues which should be implemented:

  1. Custom escape character #79
  2. Split cell to multiple properties #80
  3. Join multiple cells into one property #81
  4. Read all lines #82

Tasks like:

  1. bean in original context
  2. single facade

we can skip for now because they need additional explanation what should be done there.

I'm voting to close this issue and start working on given issues separately.

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

No branches or pull requests

3 participants