As far as I can tell, the /update/extract handler which we recently added to solrconfig.xml needs the solr-cell-*.jar archive, which we currently don't load.
So, unless I'm mistaken, if we want the extraction handler, we should also include the Solr Cell library.

CommentFileSizeAuthor
#3 2564955-3--include_solr_cell.patch5.26 KBdrunken monkey
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drunken monkey created an issue. See original summary.

Nick_vh’s picture

Nick_vh
he/him
Location Ghent
CreditAttribution: Nick_vh at Acquia commented

I'm not against including it in the main config files. Better defaults for everyone and less fuzz when wanting to make text extraction work.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey as a volunteer commented
Status: Active » Needs review
FileSize
2564955-3--include_solr_cell.patch5.26 KB

OK, good. The attached patch should do this. (Tested only with Solr 5, but the changes are the same for 3 and 4.)
This patch necessarily also includes #2564953: Fix commented-out dist/ .jar file paths, since it changes exactly the same lines.

Nick_vh’s picture

Nick_vh
he/him
Location Ghent
CreditAttribution: Nick_vh at Acquia commented

Did you test that extraction is working correctly on all versions? I checked and all versions already include /extraction/lib so that's a good sign. When talking about testing, might be good if we somehow add a testing framework to this schema. Something that can test the functionality in travis with direct queries to solr for each version?

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey as a volunteer commented

I tested only with Solr 5, sorry. Travis integration would of course help a lot here, you're right.

protools’s picture

protools CreditAttribution: protools commented

same Issue for Apache Solr Search

protools’s picture

protools CreditAttribution: protools commented

by defaults in solr 5

  <lib dir="${solr.install.dir:../../../..}/contrib/extraction/lib" regex=".*\.jar" />
  <lib dir="${solr.install.dir:../../../..}/dist/" regex="solr-cell-\d.*\.jar" />

  <lib dir="${solr.install.dir:../../../..}/contrib/clustering/lib/" regex=".*\.jar" />
  <lib dir="${solr.install.dir:../../../..}/dist/" regex="solr-clustering-\d.*\.jar" />

  <lib dir="${solr.install.dir:../../../..}/contrib/langid/lib/" regex=".*\.jar" />
  <lib dir="${solr.install.dir:../../../..}/dist/" regex="solr-langid-\d.*\.jar" />

  <lib dir="${solr.install.dir:../../../..}/contrib/velocity/lib" regex=".*\.jar" />
  <lib dir="${solr.install.dir:../../../..}/dist/" regex="solr-velocity-\d.*\.jar" />
protools’s picture

protools CreditAttribution: protools commented

and relate to https://issues.apache.org/jira/browse/SOLR-6688

section 

<admin>  
   <defaultQuery>*:*</defaultQuery>
</admin>

must be commented

protools’s picture

protools CreditAttribution: protools commented

were is dir contrib and dist should be created ? in /var/solr/data/drupal ?
ore if this is defaults dir in /opt/solr , then config must look like i wrote in post #7