-
Notifications
You must be signed in to change notification settings - Fork 25.3k
Remove [removal-of-types] docs page, and point to 7x docs #53670
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
Conversation
Pinging @elastic/es-docs (>docs) |
Pinging @elastic/es-search (:Search/Mapping) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but I will defer to @jrodewig for final review
versions of queries that perform document lookups, such as `terms`. | ||
Elasticsearch 8.0.0 no longer supports mapping types. For details on how to | ||
migrate your clusters away from mapping types, see the | ||
{ref-7x}/removal-of-types.html[removal of types] documentation for the 7x release |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{ref-7x}/removal-of-types.html[removal of types] documentation for the 7x release | |
{ref-7x}/removal-of-types.html[removal of types] documentation for the 7.x release. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One minor nit. Otherwise LGTM.
Given that types have been removed entirely in 8.0, we don't need a detailed page
explaining what they are and why they are going away. This commit replaces the
page with a short paragraph saying that types are no longer supported, and a link
to the [removal-of-types] page in 7x