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

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

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.3
    • 3.5, 4.0-ALPHA
    • clients - java
    • 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

              sarowe Steven Rowe
              dsmiley David Smiley
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: