feat(server-utils): Emit low cardinality graphql span names - #23542
Draft
andreiborza wants to merge 3 commits into
Draft
feat(server-utils): Emit low cardinality graphql span names#23542andreiborza wants to merge 3 commits into
andreiborza wants to merge 3 commits into
Conversation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Contributor
size-limit report 📦
|
4 tasks
With span streaming enabled, name graphql spans after the operation type or the graphql phase, and stop renaming the enclosing root span with the operation. Names are unchanged in static mode. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013bjBXkGkJo8eL8hkz48byi
….processing.type Follows the GraphQL OpenTelemetry Working Group, which added a dedicated attribute for the processing type rather than widening graphql.operation.type. Parse, validate and resolve spans take the static fallback name and carry the phase as an attribute instead. Claude-Session: https://claude.ai/code/session_013bjBXkGkJo8eL8hkz48byi
andreiborza
force-pushed
the
ab/graphql-low-cardinality-span-names
branch
from
August 24, 2026 19:15
f11596c to
ca3bd08
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit ca3bd08. Configure here.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
With span streaming enabled, graphql spans are now named after the operation type (
GraphQL query), orGraphQL Operationwhere the SDK has no operation type, instead of after the client-supplied operation name or resolver field path.useOperationNameForRootSpanalso no longer renames the enclosing root span. Every graphql span now carriesgraphql.processing.type, which says whether it is a parse, validate, execute or resolve span.Why
Span names must be low cardinality when span streaming is enabled, and the operation name and field path both come from the client. A low-cardinality name cannot also say which part of request processing a span covers, so
graphql.processing.typecarries that instead, following the GraphQL OpenTelemetry Working Group proposal in open-telemetry/semantic-conventions#3515. Blocked on getsentry/sentry-conventions#572, which defines the attribute; the key is inlined until that ships.Closes: #23526