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

ApacheCommonCsvRecord is redundant #244

Closed
fmbenhassine opened this issue Oct 11, 2016 · 0 comments
Closed

ApacheCommonCsvRecord is redundant #244

fmbenhassine opened this issue Oct 11, 2016 · 0 comments
Assignees
@fmbenhassine
Labels
breaking change deprecation
Milestone
v5.0.0-RC2

Comments

@fmbenhassine
Copy link
Member

ApacheCommonCsvRecord is redundant with StringRecord.

When reading a CSV file, using the FlatFileRecordReader is sufficient. There is no need for a third party library. Apache common csv may help in mapping and marshalling records.

ApacheCommonCsvRecord should be removed from the public API.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fmbenhassine fmbenhassine

Labels
breaking change deprecation
Milestone
v5.0.0-RC2
Development

No branches or pull requests
1 participant
@fmbenhassine