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

[TEST] Fix backport of #30941 #30954

Merged
merged 2 commits into from
May 30, 2018

Conversation

droberts195
Copy link
Contributor

The implementation of #30941 used functionality that is only
available in 6.4 and higher. This changes the implementation in
6.3 to one that is less elegant but still uses ML endpoints
rather than retrieving the cluster state directly.

The implementation of elastic#30941 used functionality that is only
available in 6.4 and higher.  This changes the implementation in
6.3 to one that is less elegant but still uses ML endpoints
rather than retrieving the cluster state directly.
@droberts195 droberts195 force-pushed the fix_30941_63_backport branch from 2323b5f to e08ca55 Compare May 30, 2018 09:10
@droberts195 droberts195 added >test Issues or PRs that are addressing/adding tests v6.3.0 :ml Machine learning labels May 30, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core

Copy link
Member

@jasontedor jasontedor 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 f51c497 into elastic:6.3 May 30, 2018
@droberts195 droberts195 deleted the fix_30941_63_backport branch May 30, 2018 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:ml Machine learning >test Issues or PRs that are addressing/adding tests v6.3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants