You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "jiraposter@reviews.apache.org (JIRA)" <ji...@apache.org> on 2011/08/29 06:20:45 UTC

[jira] [Commented] (HBASE-4269) Add tests and restore semantics to TableImportFormat/TableRecordReader

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

jiraposter@reviews.apache.org commented on HBASE-4269:
------------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1675/
-----------------------------------------------------------

Review request for hbase, Ted Yu and Ming Ma.


Summary
-------

commit d2bbee8696d54245b046158bacb5ab1a210e5df6
Author: Jonathan Hsieh <jo...@cloudera.com>
Date:   Sun Aug 28 19:10:57 2011 -0700

    HBASE-4269: Add tests and restore semantics to TableImportFormat/TableRecordReader
    
    Adding tests cases and restoring sematics from HBASE-4196 patch.


This addresses bug HBASE-4269.
    https://issues.apache.org/jira/browse/HBASE-4269


Diffs
-----

  src/main/java/org/apache/hadoop/hbase/mapred/TableRecordReaderImpl.java 54df2cf 
  src/main/java/org/apache/hadoop/hbase/mapreduce/TableRecordReaderImpl.java c5fc496 
  src/test/java/org/apache/hadoop/hbase/mapred/TestTableInputFormat.java PRE-CREATION 

Diff: https://reviews.apache.org/r/1675/diff


Testing
-------

New unit tests pass, running other tests.


Thanks,

jmhsieh



> Add tests and restore semantics to TableImportFormat/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.92.0, 0.90.5
>            Reporter: Jonathan Hsieh
>            Assignee: Jonathan Hsieh
>             Fix For: 0.90.5
>
>
> 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.
For more information on JIRA, see: http://www.atlassian.com/software/jira