[HDFS-8071] Redundant checkFileProgress() in PART II of getAdditionalBlock()

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.3-alpha
• Fix Version/s: 2.7.0
• Target Version/s: 2.7.0
• Hadoop Flags: Reviewed
• Assignee: Konstantin Shvachko
• Reporter: Konstantin Shvachko
• Created: 06/Apr/15 22:14
• Updated: 26/Jan/16 07:17
• Resolved: 07/Apr/15 06:56

Description

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.

Issue Links

• breaks: HDFS-9690 ClientProtocol.addBlock is not idempotent after HDFS-8071