[
https://issues.apache.org/jira/browse/HBASE-3264?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12934776#action_12934776
]
Nicolas Spiegelberg commented on HBASE-3264:
--------------------------------------------
@Todd: you're not precluded from adding Guava or whatever libraries to this,
but I don't think the default action should be to add libraries that you're not
using. Guava is currently the only dependency under addDependencyJars(Job)
that is not essential for basic HBase table operations. Since
addDependencyJars(conf, ...) allows concatenation, you can easily append jars
that are necessary for your specific config. We need to use that ourselves to
add in compression jars for HFileOutputFormat. Note that I used this api to
change the ImportTsv job to append the Guava jar, since it is the job that
requires it right now.
> 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.
