[GitHub] [lucene-solr] dsmiley opened a new pull request #2461: SOLR-2852: SolrJ: remove Woodstox dependency

[GitBox <gi...@apache.org>]

dsmiley opened a new pull request #2461:
URL: https://github.com/apache/lucene-solr/pull/2461


   It was never truly required there.
   Pervasive use of "javabin" reduces the need to care about client-side XML speed.  Better to reduce dependencies and let clients use the libs they want.
   
   https://issues.apache.org/jira/browse/SOLR-2852
   
   BTW I couldn't set "runtimeOnly" for it's inclusion in solr-core because the text tagger explicitly uses it in XmlOffsetCorrector.  I wrote that.  I don't recall if it was performance reasons or if it was for access to location metadata that I otherwise didn't see how to get.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org

[GitHub] [lucene-solr] dsmiley merged pull request #2461: SOLR-2852: SolrJ: remove Woodstox dependency

[GitBox <gi...@apache.org>]

dsmiley merged pull request #2461:
URL: https://github.com/apache/lucene-solr/pull/2461


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org