-
Notifications
You must be signed in to change notification settings - Fork 1.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
Ideas for improving the group() built-in function #884
Comments
I've mentioned in in the linked issue (#891 (comment)), but a users's question on Slack reminded me of that idea to extend this proposal, so it can help with some use cases where users don't want specific metrics measured. Specifically, we should be able to add an option/parameter to the |
Given how I have opened #3392 where we will explore making this async compatible and hopefully more generic. |
It would currently be quite difficult to implement, but ideally, once #883 is resolved, we may be able to partially override some of the global options in a
group()
call. That way we can set different tags for all metrics emitted in that group, or set local default HTTP options once #761 is implemented (especially useful for stuff like authentication).In a way, something like that already happens, since the
group
system tag's value is changed inside of thegroup()
call, but it's not flexible or user-configurable at all.The text was updated successfully, but these errors were encountered: