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

Coda Hale commented on SOLR-122:
--------------------------------

I hadn't seen Xaagly, Antonio -- I'll check it out.

libxml2 is definitely the most stable, mature XML parser for C around, though, 
and I'd be reluctant to use anything else -- XML can be tricky business (from 
security perspectives as well as performance), and libxml2 has received 
scrutiny from a much, much larger population of programmers than either Hpricot 
or Xaagly.

Either way, thanks to the well-architected nature of solrb, it's easy enough to 
write a wrapper for the XML library of your choice.

> Add optional support for Ruby-libxml2 (vs. REXML)
> -------------------------------------------------
>
>                 Key: SOLR-122
>                 URL: https://issues.apache.org/jira/browse/SOLR-122
>             Project: Solr
>          Issue Type: Improvement
>          Components: clients - ruby - flare
>            Reporter: Coda Hale
>         Attachments: libxml.rb
>
>
> This file adds drop-in support for the ruby-libxml2, which is a wrapper for 
> the libxml2 library, which is an order of magnitude or so faster than REXML.
> This depends on my SOLR-121 patch for multi-document adds, since the behavior 
> of Solr::Request::AddDocument#to_s is different.
> Requiring this makes some tests fail, but for trivial reasons: some tests are 
> directly tied to REXML, others fail due to interelement whitespace added by 
> libxml2 (which you can't disable via the Ruby interface). Functionally, it's 
> identical, and passes all functional tests.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to