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

Problems with dfs.name.edits.dirs as URI

Log workAgile BoardRank to TopRank to BottomAttach filesAttach ScreenshotBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersCreate sub-taskConvert to sub-taskMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    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

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

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

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment