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

Enis Soztutar commented on HBASE-10076:
---------------------------------------

bq. Looks like in 0.94 its just added based on the config
In trunk, initTableSnapshotMapper() calls: 
{code}
initTableMapperJob(snapshotName, scan, mapper, outputKeyClass,
        outputValueClass, job, addDependencyJars, false, 
TableSnapshotInputFormat.class);
{code}
the false is initCredentials to the overloaded function, no? I don't have 
eclipse open, I cannot check : ) 

bq. how can we test that the locality selection is correct? Its not really 
covered anywhere in this patch or the original
It is using the HDFSBlocksDistribution, which I though is tested on it's own. 
Did not check whether there is actual coverage though. It should be possible to 
mock that up I guess. 

> Backport MapReduce over snapshot files [0.94]
> ---------------------------------------------
>
>                 Key: HBASE-10076
>                 URL: https://issues.apache.org/jira/browse/HBASE-10076
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Lars Hofhansl
>            Assignee: Jesse Yates
>             Fix For: 0.94.15
>
>         Attachments: hbase-10076-v0.patch
>
>
> MapReduce over Snapshots would be valuable on 0.94.



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)

Reply via email to