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

Lease monitor may infinitely loop on the same lease

    Details

    • Type: Bug
    • Status: Open
    • Priority: Critical
    • Resolution: Unresolved
    • Affects Version/s: 2.8.0
    • Fix Version/s: None
    • Component/s: namenode
    • Labels:
      None
    • Target Version/s:

      Description

      Lease recovery incorrectly handles UC files if the last block is complete but the penultimate block is committed. Incorrectly handles is the euphemism for infinitely loops for days and leaves all abandoned streams open until customers complain.

      The problem may manifest when:

      1. Block1 is committed but seemingly never completed
      2. Block2 is allocated
      3. Lease recovery is initiated for block2
      4. Commit block synchronization invokes FSNamesytem#closeFileCommitBlocks, causing:
        • commitOrCompleteLastBlock to mark block2 as complete
        • finalizeINodeFileUnderConstruction/INodeFile.assertAllBlocksComplete to throw IllegalStateException because the penultimate block1 is "COMMITTED but not COMPLETE"
      5. The next lease recovery results in an infinite loop.

      The LeaseManager expects that FSNamesystem#internalReleaseLease will either init recovery and renew the lease, or remove the lease. In the described state it does neither. The switch case will break out if the last block is complete. (The case statement ironically contains an assert). Since nothing changed, the lease is still the “next” lease to be processed. The lease monitor loops for 25ms on the same lease, sleeps for 2s, loops on it again.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                daryn Daryn Sharp
                Reporter:
                daryn Daryn Sharp
              • Votes:
                0 Vote for this issue
                Watchers:
                11 Start watching this issue

                Dates

                • Created:
                  Updated: