Problem/Motivation
\Drupal\elasticsearch_connector\Event\SynonymsSubscriber::onAlterSettings has
$settings['analysis']['analyser']['querytime_synonyms'] = [
'type' => 'custom',
'tokenizer' => 'standard',
'filter' => ['lowercase', 'asciifolding', 'synonyms'],
];
As shown in elasticsearch documentation, the setting key here should be analyzer.
Proposed resolution
Fix the spelling of the setting name.
Remaining tasks
Write an 9.0.x patch — merge request !178 created by @fathershawn in #2
Fix test — done by @mparker17 in #3
Review and feedback — done by @mparker17 in #4
RTBC and feedback — done by @mparker17 in #4
Commit to 9.0.x — commited by @mparker17 in #5
Write an 8.0.x patch — merge request !179 created by @mparker17 in #7
Review and feedback — skipping as it is identical to the 9.0.x branch
RTBC and feedback — skipping as it is identical to the 9.0.x branch
Commit to 8.0.x — done by @mparker17 in #9
Release 9.0.x — released in 9.0.0-alpha2
Release 8.0.x — released in 8.0.0-alpha6
User interface changes
none.
API changes
none.
Data model changes
none.
Comments
Comment #4
mparker17This looks good; I updated the test, hopefully that fixes the test-failure. I've also marked this for backport.
Comment #6
mparker17Committed to 9.0.x; will move it to 8.0.x
Comment #8
mparker17Updated the issue summary
Comment #10
mparker17Committed to 8.0.x. I will update this issue when this change becomes part of a release.
Note I've created #3576581: Add a Backend test for Synonyms to help catch problems with synonyms in the future.
Comment #12
mparker17This was released in elasticsearch_connector-9.0.0-alpha2.
Comment #13
mparker17This was released in elasticsearch_connector-8.0.0-alpha6