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

[7.x] [ML] Anomaly Detection: Fix validation error when no data in index. (#86114) #86348

Merged
merged 1 commit into from
Dec 18, 2020

Conversation

walterra
Copy link
Contributor

Backports the following commits to 7.x:

…lastic#86114)

For cardinality checks, an empty index of the fields checks not returning any results would block the user from moving to the next step in the Anomaly Detection job wizard.
This PR fixes it by adding more fine grained checks and only returning warning-level messages for the above cases. A warning-level message will allow the user to continue to the next step in the wizard.
@walterra walterra self-assigned this Dec 17, 2020
@walterra walterra added the :ml label Dec 17, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
ml 7.0MB 7.0MB -409.0B
triggersActionsUi 1.5MB 1.5MB -26.9KB
total -27.3KB

Distributable file count

id before after diff
default 47558 48321 +763

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
triggersActionsUi 162.0KB 162.1KB +102.0B
Unknown metric groups

async chunk count

id before after diff
triggersActionsUi 31 32 +1

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@walterra walterra merged commit 733f7a4 into elastic:7.x Dec 18, 2020
@walterra walterra deleted the backport/7.x/pr-86114 branch December 18, 2020 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants