[ 
https://issues.apache.org/jira/browse/SOLR-2852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13136129#comment-13136129
 ] 

David Smiley commented on SOLR-2852:
------------------------------------

Two other points:
* As a general principle, minimizing client libraries/dependencies is a good 
thing, especially one that isn't likely to actually be used!
* Most people will assume all libraries in the /dist/solrj-lib/ directory are 
required when this one isn't.
* Woodstox is not the only fast implementation; some quick google searches 
unearthed [Aalto|http://wiki.fasterxml.com/AaltoHome]
                
> SolrJ doesn't need woodstox jar
> -------------------------------
>
>                 Key: SOLR-2852
>                 URL: https://issues.apache.org/jira/browse/SOLR-2852
>             Project: Solr
>          Issue Type: Improvement
>          Components: clients - java
>            Reporter: David Smiley
>            Priority: Trivial
>
> The /dist/solrj-lib/ directory contains wstx-asl-3.2.7.jar (Woodstox StAX 
> API).  SolrJ doesn't actually have any type of dependency on this library. 
> The maven build doesn't have it as a dependency and the tests pass.  Perhaps 
> Woodstox is faster than the JDK's StAX, I don't know, but I find that point 
> quite moot since SolrJ can use the efficient binary format.  Woodstox is not 
> a small library either, weighting in at 524KB, and of course if someone 
> actually wants to use it, they can.
> I propose woodstox be removed as a SolrJ dependency.  I am *not* proposing it 
> be removed as a Solr WAR dependency since it is actually required there due 
> to an obscure XSLT issue.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to