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

Xuze Yang updated HDFS-15990:
-----------------------------
    Description: 
In FSImageFormatPBSnapshot.serializeSnapshotDiffSection(), the following code 
exists:
{quote}if (i % parent.getInodesPerSubSection() == 0) {
    parent.commitSubSection(headers,
        FSImageFormatProtobuf.SectionName.SNAPSHOT_DIFF_SUB);
}{quote}
It aims to serialize SnapshotDiff information into several sub_sections(i.e. 
additional sub_sections information will be written to FileSummary Section). 
But in FSImageFormatPBSnapshot.loadSnapshotDiffSection(), it treats 
SnapshotDiffSection as a whole, rather than several sub_sections. So it's no 
need to introduce sub_sections here.

 

 

 

  was:
In FSImageFormatPBSnapshot.serializeSnapshotDiffSection(), the following code 
exists:
{quote}if (i % parent.getInodesPerSubSection() == 0){ 

        parent.commitSubSection(headers, 
FSImageFormatProtobuf.SectionName.SNAPSHOT_DIFF_SUB);

 }
{quote}
It aims to serialize SnapshotDiff information into several sub_sections(i.e. 
additional sub_sections information will be written to FileSummary Section). 
But in FSImageFormatPBSnapshot.loadSnapshotDiffSection(), it treats 
SnapshotDiffSection as a whole, rather than several sub_sections. So it's no 
need to introduce sub_sections here.

 

 

 


> No need to write to sub_section when serialize SnapshotDiff Section
> -------------------------------------------------------------------
>
>                 Key: HDFS-15990
>                 URL: https://issues.apache.org/jira/browse/HDFS-15990
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: hdfs
>    Affects Versions: 2.10.0
>            Reporter: Xuze Yang
>            Priority: Major
>
> In FSImageFormatPBSnapshot.serializeSnapshotDiffSection(), the following code 
> exists:
> {quote}if (i % parent.getInodesPerSubSection() == 0) {
>     parent.commitSubSection(headers,
>         FSImageFormatProtobuf.SectionName.SNAPSHOT_DIFF_SUB);
> }{quote}
> It aims to serialize SnapshotDiff information into several sub_sections(i.e. 
> additional sub_sections information will be written to FileSummary Section). 
> But in FSImageFormatPBSnapshot.loadSnapshotDiffSection(), it treats 
> SnapshotDiffSection as a whole, rather than several sub_sections. So it's no 
> need to introduce sub_sections here.
>  
>  
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to