Skip to content

[ML] Update error message for process update #37363

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 11, 2019
Merged

[ML] Update error message for process update #37363

merged 1 commit into from
Jan 11, 2019

Conversation

droberts195
Copy link
Contributor

When this message was first added the model debug config was
the only thing that could be updated, but now more aspects of
the config can be updated so the message needs to be more
general.
@droberts195
[ML] Update error message for process update
7078e0f 
When this message was first added the model debug config was
the only thing that could be updated, but now more aspects of
the config can be updated so the message needs to be more
general.
@droberts195 droberts195 requested a review from davidkyle January 11, 2019 13:51
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core
@droberts195
Copy link
Contributor Author

This change resolves #37349 (comment)
davidkyle
davidkyle approved these changes Jan 11, 2019
View reviewed changes
Copy link
Member

@davidkyle davidkyle 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
Copy link
Contributor Author

run gradle build tests 1
@droberts195
Copy link
Contributor Author

run gradle build tests 2
@droberts195 droberts195 merged commit 953fb93 into elastic:master Jan 11, 2019
@droberts195 droberts195 deleted the update_ml_error_message branch January 11, 2019 16:31
droberts195 added a commit that referenced this pull request Jan 11, 2019
@droberts195
[ML] Update error message for process update (#37363)
c29af26 
When this message was first added the model debug config was
the only thing that could be updated, but now more aspects of
the config can be updated so the message needs to be more
general.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :ml Machine learning v6.7.0 v7.0.0-beta1
4 participants
@droberts195 @elasticmachine @davidkyle @colings86