[
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)
