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

Redundant checkFileProgress() in PART II of getAdditionalBlock()

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.0.3-alpha
    • 2.7.0
    • None
    • None
    • 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

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

              Dates

                Created:
                Updated:
                Resolved: