-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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] AIOps: Adds cardinality check to Log Rate Analysis #181129
Conversation
f5fa13d
to
8d1730a
Compare
Pinging @elastic/ml-ui (:ml) |
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 - confirmed with the pgbench
data set with container.id
query that items with a cardinality of 1 are no longer shown.
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 ⚡
💚 Build Succeeded
Metrics [docs]
To update your PR or re-run it, just comment with: cc @walterra |
Summary
Part of #181111.
This filters field/value items from the results if the field has a cardinality of just 1 since it wouldn't be useful as a result.
Before (you can easily spot fields with cardinality of 1 because the mini histogram in the table is just one color which means the value is present in all logs):
After:
Checklist