search

smallrye / smallrye-graphql

Implementation for MicroProfile GraphQL
Apache License 2.0
160 stars 91 forks source link

Investigate using instrumentation capabilities of graphql-java for tracing #921

Open jmartisk opened 3 years ago

jmartisk commented 3 years ago

... instead of our own kind-of-instrumentation approach that we're using now.

https://www.graphql-java.com/documentation/v16/instrumentation/

t1 commented 3 years ago

This is about GraphQL/Apollo tracing, not about MP OpenTracing/Jaeger. Maybe we need both.

jmartisk commented 3 years ago

I meant we might try to use the instrumentation to hook into executions and perform our OpenTracing (and later OpenTelemetry) stuff, as it could be better than "instrumenting" via our eventing services. But I could be mistaken and it could be nonsense.