[ML] JIndex: Job exists and get job should read cluster state first.#36305
Merged
davidkyle merged 3 commits intoelastic:feature-jindex-6xfrom Dec 10, 2018
Merged
Conversation
Collaborator
|
Pinging @elastic/ml-core |
Contributor
There was a problem hiding this comment.
Are we also validating this when the job is written to the index?
Member
Author
There was a problem hiding this comment.
There are 2 checks here
- that the job id is not a group
- that any groups the job has are not job Ids
I realised the second was missing so I pushed another commit with that. Previously those checks were done in the constructor of MlMetadata which further made me realise I need to perform the same check on job update so I made that change also.
d6f7b9b to
f63f964
Compare
Member
Author
|
retest this please |
Also check the job Id is not a clusterstate group ID
f63f964 to
263ab23
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Feature branch PR
Changing methods missed in #36014 - get job and job exists.
This also adds in a missing check on PUT job that the job Id is not the same as a group Id defined in clusterstate jobs and a another check on job update that the jobs groups are not job ids.