search

slashmo / jaeger-client-swift

Jaeger Client built for Swift Distributed Tracing
Apache License 2.0
9 stars 0 forks source link

Create JaegerTracer #1

Closed slashmo closed 3 years ago

slashmo commented 3 years ago

This PR adds JaegerTracer, a Jaeger client implementing the Tracer protocol of swift-tracing.

JaegerTracer may be used with different "recording strategies", one of which is added in this PR too: ZipkinRecordingStrategy uses pure NIO (no AsyncHTTPClient) to emit flushed spans to Jaeger's Zipkin-compatible endpoint.

slashmo commented 3 years ago

Next step would be adding Samplers. I think 🤔 @ktoso

Thanks for the review. I'll merge the tracer into main and then start looking into sampling 📖

ktoso commented 3 years ago

Late LGTM, looking good :) Shaping up well