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

Olga Natkovich commented on PIG-649:
------------------------------------

+1, please, commit

> RandomSampleLoader does not handle skipping correctly in getNext()
> ------------------------------------------------------------------
>
>                 Key: PIG-649
>                 URL: https://issues.apache.org/jira/browse/PIG-649
>             Project: Pig
>          Issue Type: Bug
>    Affects Versions: types_branch
>            Reporter: Pradeep Kamath
>            Assignee: Pradeep Kamath
>             Fix For: types_branch
>
>         Attachments: PIG-649.patch
>
>
> Currently RandomSampleLoader calls skip() on the underlying input stream 
> (BufferedPositionedInputStream) in its getNext(). The input stream may not 
> actually skip over the amount the RandomSampleLoader needs in one call. 
> RandomSampleLoader should check the return value from the skip() call and 
> ensure that skip() is called repeatedly (if necessary) till the needed number 
> of bytes are skipped.

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