Remove lowercase_expanded_terms and locale from query-parser options.#20208
Merged
jpountz merged 1 commit intoelastic:masterfrom Nov 2, 2016
Merged
Conversation
This was referenced Aug 29, 2016
Contributor
There was a problem hiding this comment.
Is it really a deprecation ? Shouldn't we try to detect if the parameter is present and throw a nice exception if it happens ?
Contributor
|
Left minor comments about deprecation vs exception, LGTM otherwise |
Contributor
Author
|
@jimferenczi Thanks for having a look! The reasoning for using deprecation rather than throwing errors is to not force users to fix their queries at the same time as they upgrade the cluster, which is why I made clear in the upgrade notes that these parameters would be ignored. Since we should now do the right thing in all cases, I don't think this would cause bad surprises? |
Contributor
Fine with me, thanks for the explanation. |
174358e to
fff283d
Compare
Contributor
Author
|
I revived this pull request now that Lucene 6.3 has the bugfix for normalization and DelegatingAnalyzer (https://issues.apache.org/jira/browse/LUCENE-7429). @jimczi Would you mind having another look? |
fff283d to
9044e9b
Compare
…ons. Lucene 6.2 introduces the new `Analyzer.normalize` API, which allows to apply only character-level normalization such as lowercasing or accent folding, which is exactly what is needed to process queries that operate on partial terms such as `prefix`, `wildcard` or `fuzzy` queries. As a consequence, the `lowercase_expanded_terms` option is not necessary anymore. Furthermore, the `locale` option was only needed in order to know how to perform the lowercasing, so this one can be removed as well. Closes elastic#9978
9044e9b to
b53b848
Compare
Contributor
Author
|
@elasticmachine retest this please |
jpountz
added a commit
that referenced
this pull request
Nov 2, 2016
…ons. (#20208) Lucene 6.2 introduces the new `Analyzer.normalize` API, which allows to apply only character-level normalization such as lowercasing or accent folding, which is exactly what is needed to process queries that operate on partial terms such as `prefix`, `wildcard` or `fuzzy` queries. As a consequence, the `lowercase_expanded_terms` option is not necessary anymore. Furthermore, the `locale` option was only needed in order to know how to perform the lowercasing, so this one can be removed as well. Closes #9978
russcam
added a commit
to elastic/elasticsearch-net
that referenced
this pull request
Jun 27, 2017
Add `quote_field_suffix` to simple_query_string query Deprecate `lowercase_expanded_terms` and `locale`: elastic/elasticsearch#20208 Add "all field" execution mode to query_string and simple_query_string query: elastic/elasticsearch#20925 Fixes #2792
russcam
added a commit
to elastic/elasticsearch-net
that referenced
this pull request
Jun 29, 2017
Add `quote_field_suffix` to simple_query_string query Deprecate `lowercase_expanded_terms` and `locale`: elastic/elasticsearch#20208 Add "all field" execution mode to query_string and simple_query_string query: elastic/elasticsearch#20925 Fixes #2792
russcam
added a commit
to elastic/elasticsearch-net
that referenced
this pull request
Jun 29, 2017
* Add missing fields on simple query string query Add `quote_field_suffix` to simple_query_string query Deprecate `lowercase_expanded_terms` and `locale`: elastic/elasticsearch#20208 Add "all field" execution mode to query_string and simple_query_string query: elastic/elasticsearch#20925 Fixes #2792 * Convert to property expressions
russcam
added a commit
to elastic/elasticsearch-net
that referenced
this pull request
Jun 29, 2017
* Add missing fields on simple query string query Add `quote_field_suffix` to simple_query_string query Deprecate `lowercase_expanded_terms` and `locale`: elastic/elasticsearch#20208 Add "all field" execution mode to query_string and simple_query_string query: elastic/elasticsearch#20925 Fixes #2792 * Convert to property expressions (cherry picked from commit bef4b9d)
awelburn
pushed a commit
to Artesian/elasticsearch-net
that referenced
this pull request
Nov 6, 2017
* Add missing fields on simple query string query Add `quote_field_suffix` to simple_query_string query Deprecate `lowercase_expanded_terms` and `locale`: elastic/elasticsearch#20208 Add "all field" execution mode to query_string and simple_query_string query: elastic/elasticsearch#20925 Fixes elastic#2792 * Convert to property expressions (cherry picked from commit bef4b9d)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Lucene 6.2 introduces the new
Analyzer.normalizeAPI, which allows to applyonly character-level normalization such as lowercasing or accent folding, which
is exactly what is needed to process queries that operate on partial terms such
as
prefix,wildcardorfuzzyqueries. As a consequence, thelowercase_expanded_termsoption is not necessary anymore. Furthermore, thelocaleoption was only needed in order to know how to perform the lowercasing,so this one can be removed as well.
Closes #9978