search

open-telemetry / opentelemetry-collector

OpenTelemetry Collector
https://opentelemetry.io
Apache License 2.0
4.25k stars 1.41k forks source link

Seeking feedback on Jaeger-v2 design #8743

Open yurishkuro opened 10 months ago

yurishkuro commented 10 months ago

This is the 3rd attempt to renovate Jaeger backend components to work on top of the OTel Collector's framework.

It would be great to hear collector maintainers' feedback, concerns, suggestions.

https://github.com/jaegertracing/jaeger/issues/4843

Proposal: https://docs.google.com/document/d/1s4_6VgAS7qAVp6iEm5KYvpiGw3h2Ja5T5HpKo29iv00/view

yurishkuro commented 10 months ago

cc @codeboten on any thoughts about circular dependencies and how we can improve them

dmitryax commented 10 months ago

I posted some feedback in https://github.com/open-telemetry/opentelemetry-collector/issues/8732#issuecomment-1781916010.

About circular dependencies, is it possible to split out some modules from github.com/jaegertracing/jaeger that are used in the collector like github.com/jaegertracing/jaeger/model? Currently, it's released as one module.

yurishkuro commented 10 months ago

@dmitryax https://github.com/jaegertracing/jaeger/issues/4899