ContextDataFetcherDecorator removes TrivialDataFetcher hint

spring-projects / spring-graphql

Spring Integration for GraphQL

https://spring.io/projects/spring-graphql

Apache License 2.0

1.5k stars 297 forks source link

ContextDataFetcherDecorator removes TrivialDataFetcher hint #980

Closed osi closed 1 month ago

osi commented 1 month ago

when the ContextDataFetcherDecorator runs to decorate all defined data fetchers in the code registry, it stomps over the TrivialDataFetcher marker.

it should either:

preserve the marker, so the data fetchers do not get instrumentation applied
have a policy of not decorating trivial data fetchers (preserving them as-is)

rstoyanchev commented 1 month ago

Superseded by #983.