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

Jonathan Gray commented on HBASE-3264:
--------------------------------------

I'm fine with adding dependencies/libraries when we are using them, but in 
general think we should also aim to minimize our dependencies.

So I'm +1 for removing an additional dependency from the job if it's trivial to 
remove it.

I'm also +1 on a complete client-dep jar if we could get maven to do that for 
us.

> Remove unnecessary Guava Dependency
> -----------------------------------
>
>                 Key: HBASE-3264
>                 URL: https://issues.apache.org/jira/browse/HBASE-3264
>             Project: HBase
>          Issue Type: Bug
>          Components: mapreduce
>            Reporter: Nicolas Spiegelberg
>            Assignee: Nicolas Spiegelberg
>            Priority: Minor
>             Fix For: 0.90.1
>
>         Attachments: HBASE-3264.patch
>
>
> Currently, TableMapReduceUtil uses Guava for trivial functionality and 
> addDependencyJars() currently adds Guava by default.  However, this jar is 
> only necessary for the ImportTsv MR job.  This is annoying when naively 
> bundling hbase jar with a MR job because you now need a second dependency 
> jar.  Should default bundle with only critical dependencies and have jobs 
> that need fancy Guava functionality explicitly include them.

-- 
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