graphql-hive / graphql-hive/envelop
@envelop/opentelemetry: Change span names to reduce cardinality
- 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