openzipkin / brave

Java distributed tracing implementation compatible with Zipkin backend services.
Apache License 2.0
2.36k stars 713 forks source link

deps: reverts to zipkin-reporter 2.x #1403

Closed codefromthecrypt closed 9 months ago

codefromthecrypt commented 9 months ago

After integrating with Sleuth and Otel, it seems hastey to manage the zipkin-reporter to 3.0. The reason is both projects end up sharing classpath between Brave and reporter types. Since Brave doesn't care about this version anyway, it can leave it at the prior and remove tension. Brave 6 will completely remove zipkin version management, anyway.

Following this change, I will cut a release-5.18.1 tag, then restore the Brave 6 commit (44e4081be625d5ff73c3862e2361ae92caf83e2a) and release it.