opentelemetry-rust icon indicating copy to clipboard operation
opentelemetry-rust copied to clipboard

Split `wasm` collector from jaeger standard collector.

Open hdost opened this issue 3 years ago • 3 comments

As mentioned https://github.com/open-telemetry/opentelemetry-rust/pull/729#discussion_r805450667 We may want to look at separating the wasm implementation since it looks very different.

hdost avatar Feb 15 '22 07:02 hdost

We probably still want to reuse the part where we parse the open telemetry spans to jaeger spans so this will probably stay in opentelemtry-jaeger IMHO. To that point, we probably need to look at how to better support wasm overall. It's not urgent but I do think it's a good opportunity.

TommyCpp avatar Feb 15 '22 16:02 TommyCpp

Yea I was kinda curious if maybe there's a common crate (potentially the existing one) and then the wasm crate just has it as a dependency which does special wasm things.

hdost avatar Feb 15 '22 16:02 hdost

We have something in the API for system clock in open telemetry create. Although I don't think it currently working. https://github.com/open-telemetry/opentelemetry-rust/issues/696 need to further investigate it.

TommyCpp avatar Feb 19 '22 20:02 TommyCpp