Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-8488 HBase transitive dependencies not being pulled in when building apps like Flume which depend on HBase
  3. HBASE-9094

Rationalize dependencies; fix analysis complaints about used but non-declared dependencies

    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Closed
    • Major
    • Resolution: Duplicate
    • None
    • None
    • build
    • None

    Description

      Do a cleanup pass through our dependency set so downstreamers get good picture of what they need to include looking at pom.

      Poking with dependency plugin, found the following issues:

      + hbase-common is missing listing of a bunch of commons libs
      + Some of our classes make use of slf4j for no good reason. zk, thrift, netty, and yammer, use slf4j but no need for us to be in the slf4j managing game... so I undid our use of it and stop its transitive include everywhere.
      + hbase-examples and hbase-it do not declare includes like hbase-client, hbase-protocol, etc.
      + hbase-hadoop1-compat depended on log4j but doesn't use it.
      + hbase-prefix-tree depends on hadoop but does not declare it (uses WritiableUtil and RawComparator – just two imports... uses the Audience annotations which we could just remove but still these two critical includes)
      + hbase-server wasn't declaring its use of commons-*. Also added excludes of transitive include of log4j by zk and yammer. Got rid of slf4j as a dependency.
      + Add declarations for used libs such as httpclient and commons-math to top-level pom.
      + Removed setting versions on hadoop1 and hadoop2 profile for slf4j mess.

      Attachments

        1. dep3.txt
          19 kB
          Michael Stack
        2. dep2.txt
          18 kB
          Michael Stack
        3. dep.txt
          18 kB
          Michael Stack

        Activity

          People

            Unassigned Unassigned
            stack Michael Stack
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: