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.

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

contrib: support tracing for GraphQL #1380

Merged
merged 11 commits into from
Aug 5, 2022
Merged

contrib: support tracing for GraphQL #1380

merged 11 commits into from
Aug 5, 2022

Conversation

katiehockman
Copy link
Contributor

This change adds basic support for tracing
GraphQL functions that use github.com/99designs/gqlgen.

Query and mutation operations can be traced, with child
spans generated for reading, parsing, and validating
operation strings.

Support for subscriptions is limited to mutations that
occur within the context of a subscription, since
many subscription operations are long running (e.g. a trace
would remain open the entire time a client is subscribed to
an endpoint).

Support for field operations is not supported.

Updates #507

This change adds basic support for tracing
GraphQL functions that use github.com/99designs/gqlgen.

Query and mutation operations can be traced, with child
spans generated for reading, parsing, and validating
operation strings.

Support for subscriptions is limited to mutations that
occur within the context of a subscription, since
many subscription operations are long running (e.g. a trace
would remain open the entire time a client is subscribed to
an endpoint).

Support for field operations is not supported.

Updates #507
@katiehockman katiehockman added this to the 1.40.0 milestone Jul 12, 2022
@katiehockman katiehockman requested a review from a team July 12, 2022 20:47
@katiehockman katiehockman requested a review from a team as a code owner July 12, 2022 20:47
Copy link
Contributor

@ajgajg1134 ajgajg1134 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks pretty good so far! Just a few questions / nits here and there

contrib/99designs/gqlgen/tracer.go Outdated Show resolved Hide resolved
contrib/99designs/gqlgen/tracer.go Outdated Show resolved Hide resolved
contrib/99designs/gqlgen/tracer.go Outdated Show resolved Hide resolved
contrib/99designs/gqlgen/tracer.go Outdated Show resolved Hide resolved
ddtrace/ext/app_types.go Outdated Show resolved Hide resolved
@Hellzy Hellzy modified the milestones: 1.40.0, 1.41.0 Jul 15, 2022
ajgajg1134
ajgajg1134 previously approved these changes Aug 3, 2022
contrib/99designs/gqlgen/option.go Outdated Show resolved Hide resolved
contrib/99designs/gqlgen/tracer.go Show resolved Hide resolved
ajgajg1134
ajgajg1134 previously approved these changes Aug 5, 2022
@katiehockman katiehockman removed the request for review from knusbaum August 5, 2022 20:44
@katiehockman katiehockman merged commit 3918783 into main Aug 5, 2022
@katiehockman katiehockman deleted the katiehockman/gqlgen branch August 5, 2022 20:56
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.

None yet

4 participants