-
Notifications
You must be signed in to change notification settings - Fork 414
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
Add option to skip Introspection tracing #2610
Labels
apm:ecosystem
contrib/* related feature requests or bugs
enhancement
quick change/addition that does not need full team approval
proposal/accepted
Accepted proposals
proposal
more in depth change that requires full team approval
Comments
liamzhang40
added
the
enhancement
quick change/addition that does not need full team approval
label
Mar 12, 2024
github-actions
bot
added
the
needs-triage
New issues that have not yet been triaged
label
Mar 12, 2024
darccio
added
proposal
more in depth change that requires full team approval
apm:ecosystem
contrib/* related feature requests or bugs
proposal/accepted
Accepted proposals
and removed
needs-triage
New issues that have not yet been triaged
proposal
more in depth change that requires full team approval
labels
Mar 19, 2024
@liamzhang40 We reviewed it, and we wanted to share it's accepted. If you want to contribute, we'll be glad to review and merge your contribution. |
devin-ai-integration bot
added a commit
to conorbranagan/dd-trace-go
that referenced
this issue
May 3, 2024
devin-ai-integration bot
added a commit
to conorbranagan/dd-trace-go
that referenced
this issue
May 3, 2024
…ithout methods, addressing issues DataDog#2610 and DataDog#2684
6 tasks
@liamzhang40, I assume #2695 covers what you were looking for? |
Yup! and skipping the trivial is icing on the cake |
Glad skipping trivial doesn't just apply to our use case! 🎉 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
apm:ecosystem
contrib/* related feature requests or bugs
enhancement
quick change/addition that does not need full team approval
proposal/accepted
Accepted proposals
proposal
more in depth change that requires full team approval
Thanks for creating this tracer for gqlgen
contrib/99designs/gqlgen
.I find it unnecessary to trace the Introspection Query since it creates a huge amount of spans. Can there be an option to disable the tracing of it? Specifically the field spans
We currently wrap around
gqlTracer
and create a new tracer that looks at theOperationName
from gql context and just skips if it is aIntrospection
querySimilar to this DataDog/dd-trace-js#1046
The text was updated successfully, but these errors were encountered: