Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Database: add db.query.text as opt-in on metrics #1159

Closed
lmolkova opened this issue Jun 14, 2024 · 0 comments · Fixed by #1482
Closed

Database: add db.query.text as opt-in on metrics #1159

lmolkova opened this issue Jun 14, 2024 · 0 comments · Fixed by #1482
Labels
area:db

Comments

@lmolkova
Copy link
Contributor

lmolkova commented Jun 14, 2024 •
edited
Loading

In some applications db.query.text (templatized, sanitized) could have low enough cardinality to be reported on metrics where it can be quite useful.
We should allow users to explicitly enable it.

Update: would we also consider using query text on span name?
@lmolkova lmolkova moved this from Post Stability to Todo in Database Client Semantic Conventions Jul 12, 2024
@lmolkova lmolkova mentioned this issue Jul 12, 2024
Merged
@lmolkova lmolkova mentioned this issue Oct 15, 2024
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:db
Projects
Database Client Semantic Conventions
Status: Done
Milestone
No milestone
2 participants
@lmolkova @arminru