You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@crunch.apache.org by "Josh Wills (JIRA)" <ji...@apache.org> on 2017/01/10 16:48:58 UTC

[jira] [Commented] (CRUNCH-632) Add compression support for CSVFileSource

    [ https://issues.apache.org/jira/browse/CRUNCH-632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15815475#comment-15815475 ] 

Josh Wills commented on CRUNCH-632:
-----------------------------------

Agree that this would be worthwhile, but I'm concerned that I don't know the CSVFileSource impl well enough to do this safely (or what the minimal change would be that would satisfy this.) [~mkwhitacre], would you mind asking around? ;-)

> Add compression support for CSVFileSource
> -----------------------------------------
>
>                 Key: CRUNCH-632
>                 URL: https://issues.apache.org/jira/browse/CRUNCH-632
>             Project: Crunch
>          Issue Type: Improvement
>            Reporter: Jim McStanton
>            Priority: Minor
>
> Currently CSVFileSource does not support decompressing files before reading them, and simply opens the file and starts reading the contents: https://github.com/apache/crunch/blob/6280983179e9c690af69c2bf0e296b054122d724/crunch-core/src/main/java/org/apache/crunch/io/text/csv/CSVRecordReader.java#L127. 
> This source would more closely match TextFileSource if this support was added. The {{LineRecordReader}} supports this behavior [here|http://grepcode.com/file/repo1.maven.org/maven2/org.apache.hadoop/hadoop-mapreduce-client-core/2.7.1/org/apache/hadoop/mapreduce/lib/input/LineRecordReader.java?av=f#87]. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)