Skip to content

[7.x] [APM] Add callout to inform users of high cardinality in unique transaction names (#69112)#70153

Merged
sorenlouv merged 1 commit intoelastic:7.xfrom
sorenlouv:backport/7.x/pr-69112
Jun 28, 2020
Merged

[7.x] [APM] Add callout to inform users of high cardinality in unique transaction names (#69112)#70153
sorenlouv merged 1 commit intoelastic:7.xfrom
sorenlouv:backport/7.x/pr-69112

Conversation

@sorenlouv
Copy link
Member

Backports the following commits to 7.x:

@sorenlouv sorenlouv added the backport This PR is a backport of another PR label Jun 27, 2020
@sorenlouv sorenlouv force-pushed the backport/7.x/pr-69112 branch 12 times, most recently from dae0803 to 2066901 Compare June 27, 2020 23:27
…action names (elastic#69112)

* [APM] Add callout

Showing a callout to inform the user we have detected a high cardinality in unique transaction names and enabling them how to fix it.

* Changed color and icon

* Updated copy and styling

* Check number of returned buckets

* Add translations and docs

* Update docs link

Co-authored-by: Brandon Morelli <bmorelli25@gmail.com>

* Fix tests

Co-authored-by: Casper Hübertz <casper@formgeist.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Co-authored-by: Brandon Morelli <bmorelli25@gmail.com>
@sorenlouv sorenlouv force-pushed the backport/7.x/pr-69112 branch from 2066901 to c1f0a97 Compare June 27, 2020 23:29
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

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

@sorenlouv sorenlouv merged commit 40cc1ba into elastic:7.x Jun 28, 2020
@sorenlouv sorenlouv deleted the backport/7.x/pr-69112 branch June 28, 2020 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants