Hadoop HDFS
  1. Hadoop HDFS
  2. HDFS-4413

Secondary namenode won't start if HDFS isn't the default file system

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1-win, 1.2.1
    • Fix Version/s: 1.2.0
    • Component/s: namenode
    • Labels:
      None
    • Target Version/s:
    • Hadoop Flags:
      Reviewed

      Description

      If HDFS is not the default file system (fs.default.name is something other than hdfs://...), then secondary namenode throws early on in its initialization. This is a needless check as far as I can tell, and blocks scenarios where HDFS services are up but HDFS is not the default file system.

      1. HDFS-4413.branch-1.patch
        0.8 kB
        Mostafa Elhemali

        Activity

        Hide
        Mostafa Elhemali added a comment -

        Looking at the code this should only affect branch-1, since in trunk we secondary namenode calls NameNode.initializeGenericKeys(), which overwrites fs.default.name so it would work. I'll upload a patch for branch-1.

        Show
        Mostafa Elhemali added a comment - Looking at the code this should only affect branch-1, since in trunk we secondary namenode calls NameNode.initializeGenericKeys(), which overwrites fs.default.name so it would work. I'll upload a patch for branch-1.
        Hide
        Mostafa Elhemali added a comment -

        Patch for this in branch-1.

        Show
        Mostafa Elhemali added a comment - Patch for this in branch-1.
        Hide
        Chris Nauroth added a comment -

        +1 for the patch. Could a committer please commit this to both branch-1 and branch-1-win? I applied the patch locally on both branches and ran TestCheckpoint for a quick regression test.

        Show
        Chris Nauroth added a comment - +1 for the patch. Could a committer please commit this to both branch-1 and branch-1-win? I applied the patch locally on both branches and ran TestCheckpoint for a quick regression test.
        Hide
        Chris Nauroth added a comment -

        Also, thank you Mostafa!

        Show
        Chris Nauroth added a comment - Also, thank you Mostafa!
        Hide
        Suresh Srinivas added a comment -

        +1 for the patch.

        Show
        Suresh Srinivas added a comment - +1 for the patch.
        Hide
        Suresh Srinivas added a comment -

        I have committed this patch to branch-1, 1.2 and 1-win. Thank you Mostafa.

        Thank you Chris for reviewing the patch.

        Show
        Suresh Srinivas added a comment - I have committed this patch to branch-1, 1.2 and 1-win. Thank you Mostafa. Thank you Chris for reviewing the patch.
        Hide
        Matt Foley added a comment -

        Closed upon release of Hadoop 1.2.0.

        Show
        Matt Foley added a comment - Closed upon release of Hadoop 1.2.0.
        Hide
        Matt Foley added a comment -

        Changed the TargetVersion from 1.2.1 to 1.2.0, recognizing that it did in fact go out in 1.2.0, so should be consistent with FixVersion.

        Show
        Matt Foley added a comment - Changed the TargetVersion from 1.2.1 to 1.2.0, recognizing that it did in fact go out in 1.2.0, so should be consistent with FixVersion.

          People

          • Assignee:
            Mostafa Elhemali
            Reporter:
            Mostafa Elhemali
          • Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development