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] Data frame analytics: ensure results view loads correctly for job created in dev tools #107024
[ML] Data frame analytics: ensure results view loads correctly for job created in dev tools #107024
Conversation
Pinging @elastic/ml-ui (:ml) |
LGTM 🎉 |
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.
Looking at the docs here (https://www.elastic.co/guide/en/elasticsearch/reference/current/put-dfanalytics.html) it looks like analyzed_fields
is meant to be optional in general. Suggest to updated the interface DataFrameAnalyticsConfig
so it's optional there too. Then TS should catch the original code without the ?
.
0592c6b
to
0f1af08
Compare
Updated and ready for a final look when you get a chance 🙏 cc @walterra |
💚 Build Succeeded
Metrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
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
…b created in dev tools (elastic#107024) * check analyzed_fields exists * update DataFrameAnalyticsConfig type to make analyzed_fields optional * updating accessing analyzed_fields for type update
…b created in dev tools (elastic#107024) * check analyzed_fields exists * update DataFrameAnalyticsConfig type to make analyzed_fields optional * updating accessing analyzed_fields for type update
Summary
Fixes #106275
Checks that
analytized_fields
exists before trying to accessincludes
orexcludes
. Extends fix to outlier detection results page.Before:
After:
Checklist
Delete any items that are not applicable to this PR.