[
https://issues.apache.org/jira/browse/HDFS-8071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Konstantin Shvachko updated HDFS-8071:
--------------------------------------
Resolution: Fixed
Fix Version/s: 2.7.0
Status: Resolved (was: Patch Available)
I just committed this.
Thank you Nicholas for the prompt review.
> Redundant checkFileProgress() in PART II of getAdditionalBlock()
> ----------------------------------------------------------------
>
> Key: HDFS-8071
> URL: https://issues.apache.org/jira/browse/HDFS-8071
> Project: Hadoop HDFS
> Issue Type: Improvement
> Affects Versions: 2.0.3-alpha
> Reporter: Konstantin Shvachko
> Assignee: Konstantin Shvachko
> Fix For: 2.7.0
>
> Attachments: HDFS-8071-01.patch, HDFS-8071-02.patch,
> HDFS-8071-branch-2.7.patch
>
>
> {{FSN.getAdditionalBlock()}} consists of two parts I and II. Each part calls
> {{analyzeFileState()}}, which among other things check replication of the
> penultimate block via {{checkFileProgress()}}. See details in HDFS-4452.
> Checking file progress in Part II is not necessary, because Part I already
> assured the penultimate block is complete. It cannot change to incomplete,
> unless the file is truncated, which is not allowed for files under
> construction.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
