-
Notifications
You must be signed in to change notification settings - Fork 845
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
Remove http.target from metric attributes to control cardinality #5047
Milestone
Comments
Yep, I might not be able to finish that this week (most likely not) - the micrometer work took a lot of time to finish. Should we just remove |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
From @anuraaga's #4930 (comment):
@jsuereth I know you had wanted
http.target
as a fallback since we aren't capturinghttp.route
consistently yet, and metrics aren't nearly as useful without any route-like dimension. @mateuszrzeszutek has started the work on capturinghttp.route
(#4852) but that work may not be finished in time for 1.10.0, so we need to make a decision on what to do in 1.10.0 release (and hopefully we'll have real solution, withhttp.route
, in the next month or so).The text was updated successfully, but these errors were encountered: