[ 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.