Mailing-List: contact core-dev-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: core-dev@hadoop.apache.org
Message-ID: <1048820580.1243612605590.JavaMail.jira@brutus>
Date: Fri, 29 May 2009 08:56:45 -0700 (PDT)
From: "Raghu Angadi (JIRA)" <jira@apache.org>
To: core-dev@hadoop.apache.org
Subject: [jira] Commented: (HADOOP-5700) INode.getPathComponents throws NPE
 when given a non-absolute path
In-Reply-To: <971712353.1239994215098.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/HADOOP-5700?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12714493#action_12714493 ] 

Raghu Angadi commented on HADOOP-5700:
--------------------------------------

I didn't notice till now that this was committed. My opinion : 

-1 for runtime exception
+0.5 for IOException (with helpful message) 
+1 same as non-existent path (ie. sometimes an exception and sometimes not, upper levels decide that)

I understand how IAE helped in this particular case. we have all seen silly errors that take long time to debug sometimes. IMO what exception to be thrown should not be based on that (but the message could be made more helpful).

> INode.getPathComponents throws NPE when given a non-absolute path
> -----------------------------------------------------------------
>
>                 Key: HADOOP-5700
>                 URL: https://issues.apache.org/jira/browse/HADOOP-5700
>             Project: Hadoop Core
>          Issue Type: Bug
>          Components: dfs
>            Reporter: Todd Lipcon
>            Assignee: Todd Lipcon
>            Priority: Minor
>             Fix For: 0.21.0
>
>         Attachments: HADOOP-5700.txt
>
>
> If you pass a path that doesn't start with '/' to INode.getPathComponents, it throws a NullPointerException. Instead it should throw IllegalArgumentException to make it clear that absolute paths are required in this code.
> The attached patch fixes this, clarifies, the javadoc, and adds a test case.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.