Skip to content

increase possible metric name permutations #41

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zzantozz
Copy link
Contributor

This project was written to be able to generate metric names that were randomized in only one token of the name, which limits its usefulness in testing metric tokens at scale because all the tokens but one are the same.

In other words, even with a long name pattern like org.example.a.1.b.2.c.3.d.4.e.5.f.6.%d and your randomization factor set to 1000, you still only get 1001 tokens because all tokens leading up to the final one never change. By interspersing more randomness in that name, the number of tokens rises dramatically.

This adds the ability to do that through a new property, keeping the original behavior because tests rely on it extensively.

This project was written to be able to generate metric names that were
randomized in only one token of the name, which limits its usefulness
in testing metric tokens at scale because all the tokens but one are
the same.

In other words, even with a long name pattern like
org.example.a.1.b.2.c.3.d.4.e.5.f.6.%d and your randomization factor
set to 1000, you still only get 1001 tokens because all tokens leading
up to the final one never change. By interspersing more randomness in
that name, the number of tokens rises dramatically.

This adds the ability to do that through a new property, keeping the
original behavior because tests rely on it extensively.
@richarxt richarxt self-requested a review September 22, 2022 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant