[jira] Commented: (HADOOP-3204) LocalFSMerger needs to catch throwable

["Amar Kamat (JIRA)" <ji...@apache.org>]

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

Amar Kamat commented on HADOOP-3204:
------------------------------------

AFAIK nothing will happen if  {{FSError}} is thrown in {{LocalFSMerger}}. All the operations done in {{LocalFSMerger}} throw IOException and hence {{LocalFSMerger}} expects IOException. I think a better approach would be to make {{FSError}} extend {{IOException}} or something like that. Fixing the code to catch {{FSerror}} when {{IOException}} is expected is incorrect. Thoughts?

> LocalFSMerger needs to catch throwable
> --------------------------------------
>
>                 Key: HADOOP-3204
>                 URL: https://issues.apache.org/jira/browse/HADOOP-3204
>             Project: Hadoop Core
>          Issue Type: Bug
>          Components: mapred
>         Environment: Revision: 645751
>            Reporter: Koji Noguchi
>
> I haven't used trunk(0.17) so I don't know what would happen when FSError is thrown within LocalFSMerger thread.
> Does it have the same problem as HADOOP-3154?  

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