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

Problems with dfs.name.edits.dirs as URI

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 0.21.0
    • Fix Version/s: 0.21.0
    • Component/s: namenode
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      There are several problems with recent commit of HDFS-396.

      1. It does not work with default configuration "file:///". Throws IllegalArgumentException.
      2. ALL hdfs tests fail on Windows because "C:\mypath" is treated as an illegal URI. Backward compatibility is not provided.
      3. IllegalArgumentException should not be thrown within hdfs code because it is a RuntimException. We should throw IOException instead. This was recently discussed in another jira.
      4. Why do we commit patches without running unit tests and test-patch? This is the minimum requirement for a patch to qualify as committable, right?

        Attachments

        1. EditsDirsWin.patch
          23 kB
          Konstantin Shvachko
        2. EditsDirsWin.patch
          22 kB
          Konstantin Shvachko
        3. EditsDirsWin.patch
          19 kB
          Konstantin Shvachko
        4. failing-tests.zip
          52 kB
          Luca Telloli
        5. HDFS-456.patch
          11 kB
          Luca Telloli
        6. HDFS-456.patch
          10 kB
          Luca Telloli
        7. HDFS-456.patch
          10 kB
          Luca Telloli
        8. HDFS-456.patch
          8 kB
          Luca Telloli
        9. HDFS-456.patch
          7 kB
          Luca Telloli
        10. HDFS-456.patch
          5 kB
          Luca Telloli
        11. HDFS-456.patch
          3 kB
          Luca Telloli
        12. HDFS-456.patch
          3 kB
          Luca Telloli
        13. HDFS-456-clean.patch
          11 kB
          Jonathan Gray

          Issue Links

            Activity

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: