[
https://issues.apache.org/jira/browse/HBASE-9165?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13821987#comment-13821987
]
Ted Yu commented on HBASE-9165:
-------------------------------
{code}
+ // when making changes here, consider also mapreduce.TableMapReduceUtils
{code}
You meant TableMapReduceUtil, right ?
{code}
+ public static class ProcessContentsMapper extends MapReduceBase implements
{code}
The above class can be package-private, right ?
> Improvements to addDependencyJars
> ---------------------------------
>
> Key: HBASE-9165
> URL: https://issues.apache.org/jira/browse/HBASE-9165
> Project: HBase
> Issue Type: Sub-task
> Components: mapreduce
> Affects Versions: 0.95.2
> Reporter: Nick Dimiduk
> Assignee: Nick Dimiduk
> Fix For: 0.98.0, 0.96.1
>
> Attachments:
> 0001-HBASE-9165-mapreduce-Modularize-building-dependency-.patch,
> 0001-HBASE-9165-mapreduce-Modularize-building-dependency-.patch
>
>
> The way we support adding HBase dependencies to a MapReduce job in
> {{TableMapReduceUtils#addDependencyJars(job)}} is a bit monolithic. Advanced
> users need a way to add HBase and its dependencies to their job without us
> snooping around for ouput formats and the like (see PIG-3285). We can also
> benefit from a little more code reuse between our {{mapred}} and
> {{mapreduce}} namespaces.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
