You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "Tom White (JIRA)" <ji...@apache.org> on 2011/02/11 06:56:57 UTC

[jira] Updated: (HADOOP-7048) Wrong description of Block-Compressed SequenceFile Format in SequenceFile's javadoc

     [ https://issues.apache.org/jira/browse/HADOOP-7048?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tom White updated HADOOP-7048:
------------------------------

       Resolution: Fixed
    Fix Version/s: 0.23.0
         Assignee: Jingguo Yao
     Hadoop Flags: [Reviewed]
           Status: Resolved  (was: Patch Available)

I've just committed this. Thanks Jingguo Yao!

> Wrong description of Block-Compressed SequenceFile Format in SequenceFile's javadoc
> -----------------------------------------------------------------------------------
>
>                 Key: HADOOP-7048
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7048
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: io
>    Affects Versions: 0.21.0
>            Reporter: Jingguo Yao
>            Assignee: Jingguo Yao
>            Priority: Minor
>             Fix For: 0.23.0
>
>         Attachments: HADOOP-7048.patch
>
>   Original Estimate: 10m
>  Remaining Estimate: 10m
>
> Here is the following description for Block-Compressed SequenceFile Format in SequenceFile's javadoc:
>  * <li>
>  * Record <i>Block</i>
>  *   <ul>
>  *     <li>Compressed key-lengths block-size</li>
>  *     <li>Compressed key-lengths block</li>
>  *     <li>Compressed keys block-size</li>
>  *     <li>Compressed keys block</li>
>  *     <li>Compressed value-lengths block-size</li>
>  *     <li>Compressed value-lengths block</li>
>  *     <li>Compressed values block-size</li>
>  *     <li>Compressed values block</li>
>  *   </ul>
>  * </li>
>  * <li>
>  * A sync-marker every few <code>100</code> bytes or so.
>  * </li>
> This description misses "Uncompressed record number in the block". And "A sync-marker every few <code>100</code> bytes or so" is not the case for Block-Compressed SequenceFile Format. Correct description should be:
>  * <li>
>  * Record <i>Block</i>
>  *   <ul>
>  *     <li>Uncompressed record number in the block</li>
>  *     <li>Compressed key-lengths block-size</li>
>  *     <li>Compressed key-lengths block</li>
>  *     <li>Compressed keys block-size</li>
>  *     <li>Compressed keys block</li>
>  *     <li>Compressed value-lengths block-size</li>
>  *     <li>Compressed value-lengths block</li>
>  *     <li>Compressed values block-size</li>
>  *     <li>Compressed values block</li>
>  *   </ul>
>  * </li>
>  * <li>
>  * A sync-marker every block.
>  * </li>

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira