Skip to content
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

[ML] Remove the ML Metadata Migrator #51597

Closed
davidkyle opened this issue Jan 29, 2020 · 2 comments · Fixed by #80075
Closed

[ML] Remove the ML Metadata Migrator #51597

davidkyle opened this issue Jan 29, 2020 · 2 comments · Fixed by #80075
Assignees
Labels
:ml Machine learning v8.0.0-beta1

Comments

@davidkyle
Copy link
Member

davidkyle commented Jan 29, 2020

Prior to 6.6 anomaly detector jobs were defined in the cluster state metadata, after 6.6 all new jobs were stored in the index and extant jobs automatically migrated from cluster state to index. For those interested in the glorious history it is all in #32905.

In 8.0 all jobs must have been migrated as you cannot get to 8 without passing through 7 so the migration code and checks can be removed. This must be done before the release of 8.0.0.

The main classes to remove are MlConfigMigrator, MlConfigMigrationEligibilityCheck and the associated tests once that is done it should be possible to remove MlMetadata.

@davidkyle davidkyle added :ml Machine learning v8.0.0 labels Jan 29, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (:ml)

@droberts195
Copy link
Contributor

Closed by #80075

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:ml Machine learning v8.0.0-beta1
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants