You are viewing a plain text version of this content. The canonical link for it is here.
Posted to mapreduce-issues@hadoop.apache.org by "Hadoop QA (JIRA)" <ji...@apache.org> on 2015/03/09 21:40:39 UTC

[jira] [Commented] (MAPREDUCE-3047) FileOutputCommitter throws wrong type of exception when calling abortTask() to handle a directory without permission

    [ https://issues.apache.org/jira/browse/MAPREDUCE-3047?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14353559#comment-14353559 ] 

Hadoop QA commented on MAPREDUCE-3047:
--------------------------------------

{color:red}-1 overall{color}.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12530013/MAPREDUCE-3047-2.patch
  against trunk revision 3241fc2.

    {color:red}-1 patch{color}.  The patch command could not apply the patch.

Console output: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5246//console

This message is automatically generated.

> FileOutputCommitter throws wrong type of exception when calling abortTask() to handle a directory without permission
> --------------------------------------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-3047
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-3047
>             Project: Hadoop Map/Reduce
>          Issue Type: Improvement
>            Reporter: JiangKai
>            Priority: Trivial
>         Attachments: MAPREDUCE-3047-1.patch, MAPREDUCE-3047-2.patch, MAPREDUCE-3047.patch
>
>
> When FileOutputCommitter calls abortTask() to create a temp directory, if the user has no permission to access the directory, or a file with the same name has existed, of course it will fail, however the system will output the error information into the log file instead of throwing an exception.As a result, when the temp directory is needed later, since the temp directory hasn't been created yet, system will throw an exception to tell user that the temp directory doesn't exist.In my opinion, the exception is not exact and the error infomation will confuse users.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)