graphql-hive / graphql-hive/envelop

@envelop/opentelemetry: Change span names to reduce cardinality

Open
#2,896 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
No language data
Stars
827
Forks
132
PR merge metrics
No merged PRs in 30d

Description

**Is your feature request related to a problem? Please describe.**

The current implementation sets span names to `{parentType.name}.${fieldName}` which often leads to high cardinality.

**Describe the solution you'd like**

It would be great if @envelop/opentelemetry adopted [Semantic conventions for GraphQL server spans](https://opentelemetry.io/docs/specs/semconv/graphql/graphql-spans/)

**Describe alternatives you've considered**

**Additional context**

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in the @envelop/opentelemetry implementation where span names are formed as {parentType.name}.{fieldName}. Read the linked Semantic conventions for GraphQL server spans and compare them with the current behavior. Done means the package follows the convention and reduces high-cardinality span names.

Written by the indexing model from the issue text.

Assessment

Tech stack
graphql, typescript
Domain
api, observability
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
50/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.