Uploaded image for project: 'Hadoop HDFS'
  1. Hadoop HDFS
  2. HDFS-8071

Redundant checkFileProgress() in PART II of getAdditionalBlock()

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.3-alpha
    • Fix Version/s: 2.7.0
    • Component/s: None
    • Labels:
      None
    • Target Version/s:
    • Hadoop Flags:
      Reviewed

      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.

        Attachments

        1. HDFS-8071-branch-2.7.patch
          5 kB
          Konstantin Shvachko
        2. HDFS-8071-02.patch
          5 kB
          Konstantin Shvachko
        3. HDFS-8071-01.patch
          4 kB
          Konstantin Shvachko

          Issue Links

            Activity

              People

              • Assignee:
                shv Konstantin Shvachko
                Reporter:
                shv Konstantin Shvachko
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: