Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Solr config drupal-6.4-solr-6.x doesn't match Solr 7.1.0 #1069

Closed
kayakr opened this issue Mar 25, 2019 · 5 comments
Closed

Solr config drupal-6.4-solr-6.x doesn't match Solr 7.1.0 #1069

kayakr opened this issue Mar 25, 2019 · 5 comments
Milestone

Comments

@kayakr
Copy link
Contributor

kayakr commented Mar 25, 2019

https://github.com/Islandora-Devops/claw-playbook/blob/master/solr.yml references geerlingguy.solr that used to default to 7.1.0 but now defaults to 7.4.0 (see https://github.com/geerlingguy/ansible-role-solr/blob/master/defaults/main.yml). Either way, the solr config that gets copied across by drupal.yml is for solr 6.x (see https://github.com/Islandora-Devops/claw-playbook/blob/master/roles/internal/webserver-app/tasks/drupal.yml#L68).

Is there a particular reason for not using the drupal-6.5-solr-7.x solr config ?

@kayakr kayakr changed the title Solr config doesn't match Solr 7.1.0 Solr config drupal-6.4-solr-6.x doesn't match Solr 7.1.0 Mar 25, 2019
@seth-shaw-unlv
Copy link
Contributor

That specific line was added in November 2017 but the solr 7.x configs weren't made available by the module until June 2018 (8.x-2.0).

We now reference 8.x-2.0, so it should be a simple PR changing the "6" to a "7" to fix that.

@kayakr
Copy link
Contributor Author

kayakr commented Mar 26, 2019

@seth-shaw-unlv Thanks. FWIW, the solr 7.x configs are working ok on my local instance with Search API Solr 8.x-2.6.

I had to change /var/solr/data/CLAW/conf/solrcore.properties to set solr.install.dir=/opt/solr/.

Contrary to https://www.drupal.org/project/search_api_solr/issues/3005186#comment-12807969 I had to change schema.xml _version_ to
<field name="_version_" type="plong" indexed="true" stored="true"/>

@whikloj whikloj added this to the 1.0.0 milestone Apr 11, 2019
@Natkeeran
Copy link
Contributor

Natkeeran commented May 22, 2019

@kayakr
Can you please provide some additional context about the following change (i.e why is this required).

<field name="_version_" type="plong" indexed="true" stored="true"/>

@kayakr
Copy link
Contributor Author

kayakr commented May 22, 2019

@Natkeeran It was a while ago now, but when I tried to reload the Solr core with the 7.x config it reported a Java exception: Caused by: java.lang.IllegalStateException: unexpected docvalues type NONE for field '_version_' (expected=NUMERIC). Re-index with correct docvalues type."

@seth-shaw-unlv
Copy link
Contributor

@kayakr and @Natkeeran, it doesn't look like loading the 7.x config at install needs the field change. My testing for Islandora-Devops/islandora-playbook/pull/107 didn't show any issues and the SOLR logs didn't show any errors.

Feel free to re-open the ticket if it pops up again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants