[ 
https://issues.apache.org/jira/browse/PIG-1197?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12802924#action_12802924
 ] 

Pradeep Kamath commented on PIG-1197:
-------------------------------------

Alan is right - TextLoader on the load-store-redesign branch already uses 
TextInputFormat (and hence LineReader) - do committers feel this patch is 
important enough that it should be committed to trunk? Otherwise I would vote 
in favor of just keeping it a patch as Alan suggested for people to use since 
TextLoader probably is not a frequently used Loader (am guessing).

> TextLoader should be updated to match changes to PigStorage
> -----------------------------------------------------------
>
>                 Key: PIG-1197
>                 URL: https://issues.apache.org/jira/browse/PIG-1197
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>    Affects Versions: 0.6.0
>            Reporter: Alan Gates
>            Assignee: Alan Gates
>            Priority: Minor
>             Fix For: 0.7.0
>
>         Attachments: PIG-1197.patch
>
>
> In 0.6 PigStorage was changed to use LineRecordReader to parse lines out of 
> its stream instead of doing the parsing itself.  This resulted in about a 30% 
> speed up in parsing time.  TextLoader should be changed to use 
> LineRecordReader in the same way to benefit from the same speed up.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to