-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use gRPC #673
Comments
Just to make it explicit: based on what was talked on the last week's Jaeger meeting, this is not about removing Thrift, but about adding gRPC. |
See jaegertracing/jaeger-idl#41 for potential first steps. |
Would love to see that happen. Took me a couple of hours to generate jaeger-idl for .NET Core since the code generator for 0.9.3 does not include .NET Core but the NuGET packages are all 0.9.3 newest... finally found a working package on GitHub (not NuGET...) gRPC would have been a lot easier, and I already have the runtime included. Would allow me to remove thrift runtime. |
Superseded by #773 |
User gRPC between agent and collector. We could also consider using it in client libraries to report data.
See:
https://groups.google.com/forum/#!topic/jaeger-tracing/toBNkL_zEDE
#526 (comment)
The text was updated successfully, but these errors were encountered: