This repository has been archived by the owner on Sep 14, 2022. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 33
Implement opentracing #82
Comments
+1 |
Can you give me the motivation/reasons why do you need this support? |
+1 this would also be helpful for us. We are building gRPC services and opentracing API plays nicely with gRPC. Currently we use a very convoluted combination of opentracing API + grpc integration -> zipkin tracer (Brave in Java) -> stackdriver-zipkin-connector -> stackdriver. |
I would like this also, as it lets me work against a standard API, rather than a vendor specific one. |
I would like this also, as it lets me work against a standard API, rather
than a vendor specific one.
There are a few vendors working on OpenTracing, but it is by far away from
an industry standard.
If you'd like an OpenTracing bridge, I'd recommend asking them to make one
in opentracing-contrib. If OpenTracing is indeed vendor neutral, you'll
have no problem mapping to census' tracing api. There's no good reason to
create burden here. This is a core repo and the pace of api change here is
completely separate from the pace of change in OptenTracing.
|
+1 this would also be helpful for us. We are building gRPC services and
opentracing API plays nicely with gRPC.
You do know that census is the native tracing implementation for grpc,
right? :) I would be very surprised if a third-party tracing implementation
worked better, but it is possible
https://github.com/grpc/grpc-java/blob/master/core/src/main/java/io/grpc/internal/CensusTracingModule.java
|
We use OpenTracing with Jaeger with GRPC, ActiveMQ and REST all linked together. Census is dead. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
What do you think about make it an implementation of opentracing ?
The interfaces are defined here: https://github.com/opentracing/opentracing-java
The text was updated successfully, but these errors were encountered: