Remove analyzer version deprecation check#118167
Remove analyzer version deprecation check#118167elasticsearchmachine merged 4 commits intoelastic:mainfrom
Conversation
|
Pinging @elastic/es-search-relevance (Team:Search Relevance) |
john-wagster
left a comment
There was a problem hiding this comment.
This is a little hard to validate in review, a lot of things touched. But what I'm seeing here LGTM.
|
@elasticmachine update branch |
|
@elasticmachine update branch |
|
Re-visiting this PR for brief pulse check if we still want the deprecation warning for analysis components that use the "version" parameter. With the read-only N-2 index support we might encounter indices that use this setting in their analysis configuration. As far as I see we didn't do anything with it at least in most of the 7x releases, with the current state we would simply ignore the settings and be done with it. |
|
@cbuescher I am fine keeping it removed these settings were deprecated in 7x, which is |
Version has been deprecated since v7: #74073
Removing checking for the version setting. It has been ignored and does nothing for the entirety of 8 and for the last minors of v7.