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

Clarify usage of the "solr path" setting in implementation details #3261

Merged
merged 4 commits into from
Sep 24, 2021

Conversation

damienmckenna
Copy link
Contributor

@damienmckenna damienmckenna commented Sep 24, 2021

The Problem/Issue/Bug:

Different systems use Solr API v1 and others use v2.

How this PR Solves The Problem:

Noted that the "Solr path" setting must be set to "/solr" when using Solr API v1 and just "/" when using Solr API v2.

Manual Testing Instructions:

Automated Testing Overview:

Related Issue Link(s):

https://drupal.slack.com/archives/C5TQRQZRR/p1632433128297900

Release/Deployment notes:

The default "Solr path" value from Search API Solr is incorrect, it defaults to "/solr", which doesn't work, it has to be just "/".
This can be an issue when changing from one Solr system to another, e.g. upgrading from v3 to v7 and from Drupal's Search API Solr module v1 to v4.
Noted the correct "Solr path" setting to use
Copy link
Member

@rfay rfay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@rfay rfay merged commit 59062ad into ddev:master Sep 24, 2021
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

Successfully merging this pull request may close these issues.

None yet

2 participants