Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-2756

SolrJ maven dependencies are faulty; needless dependency on lucene-core

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.3
    • Fix Version/s: 3.5, 4.0-ALPHA
    • Component/s: clients - java
    • Labels:
      None

      Description

      I included a SolrJ 3.3 dependency into a new project and I noticed needless dependencies transitive show up.

      Here is a subset of the output from mvn dependency:tree:

      [INFO] +- org.apache.solr:solr-solrj:jar:3.3.0:compile
      [INFO] |  +- org.apache.lucene:lucene-core:jar:3.3.0:compile
      [INFO] |  +- commons-httpclient:commons-httpclient:jar:3.1:compile
      [INFO] |  |  \- commons-codec:commons-codec:jar:1.2:compile
      [INFO] |  +- org.apache.geronimo.specs:geronimo-stax-api_1.0_spec:jar:1.0.1:compile
      [INFO] |  +- org.apache.zookeeper:zookeeper:jar:3.3.1:compile
      [INFO] |  |  +- log4j:log4j:jar:1.2.15:compile
      [INFO] |  |  |  \- javax.mail:mail:jar:1.4:compile
      [INFO] |  |  |     \- javax.activation:activation:jar:1.1:compile
      [INFO] |  |  \- jline:jline:jar:0.9.94:compile
      [INFO] |  \- org.codehaus.woodstox:wstx-asl:jar:3.2.7:compile
      [INFO] |     \- stax:stax-api:jar:1.0.1:compile
      

      Clearly there is an inconsistency with solr/dist/solrj-lib and this list.

      • lucene-core dependency should be removed
      • AFAIK, geronimo-stax-api and wstx-asl are only needed for Java 1.5. Right? These can be put in a maven profile activated by jdk1.5.
      • zookeeper dependency should be removed. Is this used in Solr 4? Even if it is, it strikes me as an optional dependency.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                steve_rowe Steve Rowe
                Reporter:
                dsmiley David Smiley
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: