search

jaegertracing / jaeger

CNCF Jaeger, a Distributed Tracing Platform
https://www.jaegertracing.io/
Apache License 2.0
20.52k stars 2.44k forks source link

Remove zipkin thrift endpoint from agent #1186

Open pavolloffay opened 6 years ago

pavolloffay commented 6 years ago

Remove 5775/udp which accepts zipkin.thrift over compact thrift protocol

From https://github.com/jaegertracing/jaeger/pull/1181#discussion_r233632594

so nobody outside of Uber should be using those old clients. If so, we can remove that endpoint and re-keep it in our internal build only.

yurishkuro commented 6 years ago

I would like to validate this internally first (we should have metrics).