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

David Alves commented on HBASE-581:
-----------------------------------

Hi.
The main reason I couldn't subclass TIF is that HTable instance in TIF is 
package protected.
I then implemented an inner FilteredTableRecordReader that *can* receive a 
RowFilterInterface as argument, as this is not mandatory (the filter can be 
null) this could be a nice addition even to the HBase TIF, receiveing the 
filter class (and someway it's contructor arguments) by configuration. Even if 
this is not possible just allowing to set the record reader instance in TIF 
would make Overriding getRecordReader (and the TableSplit casts) unecessary.

> Allow adding filters to TableInputFormat (At same time, ensure TIF and TOF 
> are subclassable)
> --------------------------------------------------------------------------------------------
>
>                 Key: HBASE-581
>                 URL: https://issues.apache.org/jira/browse/HBASE-581
>             Project: Hadoop HBase
>          Issue Type: Improvement
>            Reporter: stack
>            Priority: Critical
>             Fix For: 0.2.0
>
>


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

Reply via email to