-
Notifications
You must be signed in to change notification settings - Fork 155
Update the index compatibility information #1617
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
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! 🚀
Very nice!
Found more details about the index compatibility version checks in #118941 and updated the docs to reflect that.
The changes in PR#118941 provide additional context on the compatibility. |
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.
approved with one thing to fix and a couple of suggestions
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.
🏁 apart from if you want to drop the footnote links!
These changes are part of the peer review suggestions as well.
Fixes #1200
Updates the Index compatibility table in the Snapshot and restore section of the deployment docs.
Preview available here
These changes are confirmed by eng and also PR#118941 provides additional context on the compatibility.