jtracer from jaeger-v2 should be removed #7537
Closed
AnmolxSingh
started this conversation in
General
Replies: 2 comments
-
yes, but please open an issue, not a discussion |
Beta Was this translation helpful? Give feedback.
0 replies
-
Issue opened for the respective concern at #7540 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
In the Jaeger-V2 codebase we use jtracer as the OTEL collector earlier did not initialize the tracer.
jaeger/cmd/jaeger/internal/extension/jaegerquery/server.go
Line 65 in f135545
But as of now the respective github issue seems to be completed
open-telemetry/opentelemetry-collector#7532
So it seems fair to remove jtracer so as not to trace the service twice.
Beta Was this translation helpful? Give feedback.
All reactions