Skip to content

[8.0] [ML] Set model state compatibility version to 8.0.0#2140

Merged
droberts195 merged 1 commit intoelastic:8.0from
droberts195:increment_model_state_version_80
Nov 25, 2021
Merged

[8.0] [ML] Set model state compatibility version to 8.0.0#2140
droberts195 merged 1 commit intoelastic:8.0from
droberts195:increment_model_state_version_80

Conversation

@droberts195
Copy link
Copy Markdown

Setting this for the new major version gives us a nice
clean break for model state compatibility. Model snapshots
generated by 8.0 will not be loadable by 7.x nodes, and
in the version where we eventually stop supporting upgrades
from 7.x to the latest version we'll be able to remove all
the model state backwards compatibility code related to
version 7.x.

Backport of #2139

Setting this for the new major version gives us a nice
clean break for model state compatibility.  Model snapshots
generated by 8.0 will not be loadable by 7.x nodes, and
in the version where we eventually stop supporting upgrades
from 7.x to the latest version we'll be able to remove all
the model state backwards compatibility code related to
version 7.x.

Backport of elastic#2139
Copy link
Copy Markdown
Contributor

@edsavage edsavage left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@droberts195 droberts195 merged commit 19897f1 into elastic:8.0 Nov 25, 2021
@droberts195 droberts195 deleted the increment_model_state_version_80 branch November 25, 2021 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants