hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Wei-Chiu Chuang (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-13824) FsShell can suppress the real error if no error message is present
Date Sun, 20 Nov 2016 20:11:58 GMT

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

Wei-Chiu Chuang commented on HADOOP-13824:

[~jzhuge] thanks for the patch. You probably don't need this line in the test:
Mockito.when(mockCommand.getDescription()).thenReturn("A mock command");
Other than that, the patch looks good to me.

> FsShell can suppress the real error if no error message is present
> ------------------------------------------------------------------
>                 Key: HADOOP-13824
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13824
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 2.7.1, 2.7.3
>            Reporter: Rob Vesse
>            Assignee: John Zhuge
>              Labels: supportability
>         Attachments: HADOOP-13824.001.patch
> The {{FsShell}} error handling assumes in {{displayError()}} that the {{message}} argument
is not {{null}}. However in the case where it is this leads to a NPE which results in suppressing
the actual error information since a higher level of error handling kicks in and just dumps
the stack trace of the NPE instead.
> e.g.
> {noformat}
> Exception in thread "main" java.lang.NullPointerException
> 	at org.apache.hadoop.fs.FsShell.displayError(FsShell.java:304)
> 	at org.apache.hadoop.fs.FsShell.run(FsShell.java:289)
> 	at org.apache.hadoop.util.ToolRunner.run(ToolRunner.java:70)
> 	at org.apache.hadoop.util.ToolRunner.run(ToolRunner.java:84)
> 	at org.apache.hadoop.fs.FsShell.main(FsShell.java:340)
> {noformat}
> This is deeply unhelpful because depending on what the underlying error was there may
be no stack dumped/logged for it (as HADOOP-7114 provides) since {{FsShell}} doesn't explicitly
dump traces for {{IllegalArgumentException}} which appears to be the underlying cause of my
issue.  Line 289 is where {{displayError()}} is called for {{IllegalArgumentException}} handling
and that catch clause does not log the error.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org

View raw message