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

        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: