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

Jonathan Hsieh commented on HBASE-4269:
---------------------------------------

Jan, 

When I did this patch, the two versions had different error recovery path and I 
made them similar.  UnknownScannerException is a subclass DoNotRetryIOException 
so I chose that instead.  

I'm assuming this is causing some pain now -- how is this affecting the job you 
are running? (is it catching and rethrowing other exceptions as well?) 

If there is something we need to change I'm fine with that.  Let's file a new 
issue -- this patch has been in included in a few releases now. 
                
> Add tests and restore semantics to TableInputFormat/TableRecordReader
> ---------------------------------------------------------------------
>
>                 Key: HBASE-4269
>                 URL: https://issues.apache.org/jira/browse/HBASE-4269
>             Project: HBase
>          Issue Type: Improvement
>          Components: mapred, mapreduce, test
>    Affects Versions: 0.90.5, 0.92.0
>            Reporter: Jonathan Hsieh
>            Assignee: Jonathan Hsieh
>             Fix For: 0.90.5
>
>         Attachments: 
> 0001-HBASE-4269-Add-tests-and-restore-semantics-to-TableI.patch, 
> 0001-HBASE-4269-Add-tests-and-restore-semantics-to-TableI.patch
>
>
> HBASE-4196 Modified the semantics of failures in 
> TableImportFormat/TableRecordReader, and had no tests cases.  This patch 
> restores semantics to rethrow when a DoNotRetryIOException is triggered and 
> adds test cases.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to