-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
chore(tag_cardinality_limit transform): Add multi-value metric tag support to the tag_cardinality_limit
transform
#15505
chore(tag_cardinality_limit transform): Add multi-value metric tag support to the tag_cardinality_limit
transform
#15505
Conversation
The current tag value set represents only strings, ie tags with a value. This adds support for "bare" tags without a value. All existing interfaces stay the same (or equivalent) with this change, so none of the bare tags are actually exposed yet to users of metric tags.
✅ Deploy Preview for vrl-playground ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for vector-project canceled.
|
Regression Test Results
Run ID: c74b192a-62fd-465e-860f-330fbfe817cd Explanation
A regression test is an integrated performance test for
The table below, if present, lists those experiments that have experienced a
statistically significant change in their No interesting changes in Fine details of change detection per experiment.
|
ref: #14742
This adds muti-value tag support. After some discussions with @bruceg, we decided to only implement one of the 3 modes discussed in the RFC, potentially expanding it in the future. The mode implemented treats a single "value" (as counted towards the cardinality limit) as the unique set of values on a tag.