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

Todd Lipcon commented on HBASE-3264:
------------------------------------

I'm sort of -0 on this. I think guava is really useful, and in general, we 
should be willing to add libraries if they make our lives easier.

Perhaps a better solution would be to package some kind of 
hbase-client-dependencies.jar with all of our dependencies?

> 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