[ 
https://issues.apache.org/jira/browse/HADOOP-6196?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12751808#action_12751808
 ] 

Jay Booth commented on HADOOP-6196:
-----------------------------------

I agree, your fix seems relatively foolproof and sidesteps the version bump 
entirely.  I've tested it pretty thoroughly in pseudo distributed and will be 
giving it a full workout over the next few days, in case that helps with 
anyone's confidence.  I'd really like to see this go into 0.21, if possible, 
even if sync(0) isn't very common the class should still adhere to contract.

> sync(0); next() breaks SequenceFile
> -----------------------------------
>
>                 Key: HADOOP-6196
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6196
>             Project: Hadoop Common
>          Issue Type: Bug
>            Reporter: Jay Booth
>            Assignee: Jay Booth
>             Fix For: 0.21.0
>
>         Attachments: C6196-0.patch, C6196-test.patch, 
> hadoop-6196-multipatch.txt, sync-bug.patch
>
>
> Currently, the end of the SequenceFile header is a sync block that isn't 
> prefaced with SYNC_ESCAPE.  This means that sync(0) followed by next() fails.

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