-
Notifications
You must be signed in to change notification settings - Fork 24.6k
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] Set type on index requests in 6.x #36850
[ML] Set type on index requests in 6.x #36850
Conversation
This is a followup to elastic#36796 that is only required for 6.x. The type must be specified in the index request. This is not necessary in 7.x, hence the functionality works in the master build.
Pinging @elastic/ml-core |
Marked as |
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
run gradle build tests 2 |
It seems that the changes of #36856 need to be merged into this PR before it will pass CI |
run default distro tests |
run gradle build tests 1 |
run default distro tests |
run gradle build tests 1 |
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.
Simple enough :). Sorry for not doing this myself :)
run gradle build tests 1 |
This is a followup to #36796 that is only required for 6.x. The type must be specified in the index request. This is not necessary in 7.x, hence the functionality works in the master build.
This is a followup to #36796 that is only required
for 6.x. The type must be specified in the index
request. This is not necessary in 7.x, hence the
functionality works in the master build.