Hadoop HDFS
  1. Hadoop HDFS
  2. HDFS-301

Provide better error messages when fs.default.name is invalid

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:

      Description

      this the followon to HADOOP-5687 - its not enough to detect bad uris, we need good error messages and a set of tests to make sure everything works as intended.

        Issue Links

          Activity

          Steve Loughran created issue -
          Steve Loughran made changes -
          Field Original Value New Value
          Attachment HADOOP-5095-1.patch [ 12408889 ]
          Steve Loughran made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Steve Loughran made changes -
          Link This issue relates to HADOOP-5687 [ HADOOP-5687 ]
          Steve Loughran made changes -
          Link This issue relates to HADOOP-5200 [ HADOOP-5200 ]
          Steve Loughran made changes -
          Link This issue relates to HADOOP-5140 [ HADOOP-5140 ]
          Hide
          Steve Loughran added a comment -

          I've submitted a patch that is simply a rename of the tests/changes attached to HADOOP-5867; the filename may be different, but not the code

          Show
          Steve Loughran added a comment - I've submitted a patch that is simply a rename of the tests/changes attached to HADOOP-5867 ; the filename may be different, but not the code
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12408889/HADOOP-5095-1.patch
          against trunk revision 778994.

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 3 new or modified tests.

          -1 patch. The patch command could not apply the patch.

          Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-vesta.apache.org/407/console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12408889/HADOOP-5095-1.patch against trunk revision 778994. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 3 new or modified tests. -1 patch. The patch command could not apply the patch. Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-vesta.apache.org/407/console This message is automatically generated.
          Owen O'Malley made changes -
          Project Hadoop Common [ 12310240 ] HDFS [ 12310942 ]
          Key HADOOP-5905 HDFS-301
          Affects Version/s 0.21.0 [ 12313563 ]
          Component/s dfs [ 12310710 ]
          Hide
          dhruba borthakur added a comment -

          @Steve: this patch does not merge with hdfs trunk anymore.

          Show
          dhruba borthakur added a comment - @Steve: this patch does not merge with hdfs trunk anymore.
          dhruba borthakur made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Steve Loughran made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Hide
          Steve Loughran added a comment -

          I'll work on this again. Move the tests to JUnit4 too

          Show
          Steve Loughran added a comment - I'll work on this again. Move the tests to JUnit4 too
          Hide
          Steve Loughran added a comment -

          This is going to be a fun patch to nurture through

          • Some of the error handling is in src/core/org/apache/hadoop/fs/FileSystem.java, in -common
          • More error handling is in the Namenode, patches and testing go into -hdfs

          I propose splitting the two

          1. -common code can have its own test, go into common
          2. Namenode patches can go in, test code moved to Junit4

          The the second patch depends on the first, or at least its tests do; there's no compile-time dependencies

          Show
          Steve Loughran added a comment - This is going to be a fun patch to nurture through Some of the error handling is in src/core/org/apache/hadoop/fs/FileSystem.java, in -common More error handling is in the Namenode, patches and testing go into -hdfs I propose splitting the two -common code can have its own test, go into common Namenode patches can go in, test code moved to Junit4 The the second patch depends on the first, or at least its tests do; there's no compile-time dependencies
          Gavin made changes -
          Reporter Steve Loughran [ steve_l ] Steve Loughran [ stevel@apache.org ]
          Gavin made changes -
          Assignee Steve Loughran [ steve_l ] Steve Loughran [ stevel@apache.org ]
          Steve Loughran made changes -
          Status In Progress [ 3 ] Open [ 1 ]
          Allen Wittenauer made changes -
          Labels newbie
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Patch Available Patch Available
          1m 19s 1 Steve Loughran 23/May/09 21:38
          Patch Available Patch Available Open Open
          38d 20h 46m 1 dhruba borthakur 01/Jul/09 18:25
          Open Open In Progress In Progress
          175d 3h 39m 1 Steve Loughran 23/Dec/09 21:04
          In Progress In Progress Open Open
          408d 23h 17m 1 Steve Loughran 05/Feb/11 20:22

            People

            • Assignee:
              Steve Loughran
              Reporter:
              Steve Loughran
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:

                Development