tez-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bikas Saha (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (TEZ-1359) SimpleMRProcessor commit should handle IO/Interrupted exceptions
Date Wed, 22 Oct 2014 18:36:33 GMT

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

Bikas Saha commented on TEZ-1359:
---------------------------------

I just looked at the code in the project when I tried to apply the patch. It looks like it
has already been updated to catch IOExceptions and it seems IO exceptions should not be retried
by us because we dont know what exactly the real commit user code is doing. We could have
handled interrupted exception via this patch, but we recently decided not to catch interrupted
exceptions in the task execution and instead let them bubble to the top.
Given the above I am going to resolve this jira as wont-fix. I should have done this earlier
so that your time would not have been wasted in creating the patch. I am sorry about that.
Please consider contributing to some other jiras. Your help is much appreciated.

> SimpleMRProcessor commit should handle IO/Interrupted exceptions
> ----------------------------------------------------------------
>
>                 Key: TEZ-1359
>                 URL: https://issues.apache.org/jira/browse/TEZ-1359
>             Project: Apache Tez
>          Issue Type: Bug
>    Affects Versions: 0.6.0
>            Reporter: Bikas Saha
>              Labels: newbie
>         Attachments: TEZ-1359-20141021.patch, TEZ-1359-20141022.patch
>
>
> The commit should have some retries against RPC IO exceptions. Interrupted exceptions
should abort() and complete.



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

Mime
View raw message