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

Problems with dfs.name.edits.dirs as URI

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • 0.21.0
    • 0.21.0
    • namenode
    • None
    • 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

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

              Dates

                Created:
                Updated:
                Resolved: