Per the comment in #2949562: Word stemming breaks the query string, it would be useful if the UI provided a note that the custom stemmer logic is not needed with advanced index systems, like Solr.

CommentFileSizeAuthor
#4 2949899-4--processors_solr_warning.patch2.26 KBdrunken monkey
#4 2949899-4--processors_solr_warning--interdiff.txt678 bytesdrunken monkey
#2 2949899-2--processors_solr_warning.patch2.26 KBdrunken monkey
#2 screenshot.png22.36 KBdrunken monkey

Comments

DamienMcKenna created an issue. See original summary.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Category: Task » Feature request
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new screenshot.png22.36 KB
new 2949899-2--processors_solr_warning.patch2.26 KB

We currently don't do this for any of the other processors, either, and probably most of them aren't for use with Solr. So adding a note just to the Stemmer would be more confusing than helpful, I think.
When creating/editing a Solr server, the backend plugin description already contains the following, though:

Will use internal Solr preprocessors, so Search API preprocessors should for the most part be deactivated.

But it's likely that this won't be enough for a lot of users, so yes, adding an additional note might be a good idea.
Maybe something like this?

Select processors which will pre- and post-process data at index and search time, and their order. Most processors will only influence fulltext fields, but refer to their individual descriptions for details regarding their effect. Also, some processors shouldn't be used with more advanced search engines (like Solr or Elasticsearch), since the search engine already provides this functionality. Check the server's service class description for details.

In any case, good idea, thanks!

Status: Needs review » Needs work

The last submitted patch, 2: 2949899-2--processors_solr_warning.patch, failed testing. View results

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Needs work » Needs review
StatusFileSize
new 2949899-4--processors_solr_warning--interdiff.txt678 bytes
new 2949899-4--processors_solr_warning.patch2.26 KB

Oops.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented

Could you please just review and tell me whether the addition looks good to you, or whether you have a better suggestion?

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Reviewed & tested by the community

Yes, I think this is a great improvement, thank you.

  • drunken monkey committed 5f46652 on 7.x-1.x 
    Issue #2949899 by drunken monkey, DamienMcKenna: Added a warning against...
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Reviewed & tested by the community » Fixed

Good to hear, thanks for reviewing!
Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.