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

ApacheCommonCsvRecordReader is redundant #245

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

ApacheCommonCsvRecordReader is redundant #245

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

This is related to #244 , with the removal of ApacheCommonCsvRecord, the ApacheCommonCsvRecordReader makes no sense. This reader is redundant with FlatFileRecordReader.

The ApacheCommonCsvRecordReader should be removed from the public API.
@fmbenhassine fmbenhassine added this to the 5.0.0-RC2 milestone Oct 11, 2016
@fmbenhassine fmbenhassine self-assigned this Oct 11, 2016
@fmbenhassine fmbenhassine changed the title ApacheCommonCsvRecordReader is redundent ApacheCommonCsvRecordReader is redundant Oct 11, 2016
fmbenhassine added a commit that referenced this issue Oct 11, 2016
@fmbenhassine
issue #245 : remove redundant ApacheCommonCsvRecordReader API
872ca6e
fmbenhassine added a commit that referenced this issue Oct 28, 2016
@fmbenhassine
issue #245 : add MultiFileRecordReader
be8173e
fmbenhassine added a commit that referenced this issue Feb 15, 2017
@fmbenhassine
issue #245 : add MultiFileRecordReader
b7d79dd
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