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

Update Binary format in the Specification

Open carlosalberto opened this issue 5 years ago • 11 comments

As https://github.com/open-telemetry/opentelemetry-specification/pull/426 is temporarily removing Binary format, we need to make sure Binary is re-added and updated properly with the OTEP 66 updates.

Assigning for now to @MikeGoldsmith but we can discuss who can tackle this item going forward.

carlosalberto avatar Feb 04 '20 19:02 carlosalberto

Added @carlosalberto to the assignees since he promised to add it back :)

bogdandrutu avatar Feb 06 '20 19:02 bogdandrutu

That works :)

carlosalberto avatar Feb 06 '20 19:02 carlosalberto

Can we assign a milestone for this issue? Any release that wants to offer a full OpenTracing or OpenCensus compatibility layer will need this feature.

Oberon00 avatar Mar 24 '20 17:03 Oberon00

Also, we need that feature at Dynatrace and will have to stay on an older version of the Java API until this is fixed.

Oberon00 avatar Mar 24 '20 17:03 Oberon00

@Oberon00 So the milestone would be 0.5.0 - for which we don't have a date for yet (other than being the next important milestone). Maybe we need to discuss this in the next SIG/community call, to plan what/how happens after beta ;)

carlosalberto avatar Mar 24 '20 19:03 carlosalberto

Talked with @carlosalberto today and based on the history of the intention of binary format (it looks like it was only desired to be temporarily removed), looks like this one is important to get in sooner because it can cause backward breaking change. Therefore, changing priority to p1.

andrewhsu avatar Aug 05 '20 14:08 andrewhsu

from the spec issue triage mtg today, discussed this and moving to after ga

andrewhsu avatar Sep 18 '20 16:09 andrewhsu

Documentation states that the binary propagator support is upcoming, however this thread has been stale for almost 4 years. Is this still planned?

annabokhan avatar Mar 08 '24 16:03 annabokhan

+1 to above question. It has been 4 years, since no activity. If there are any prior investigations why it was no longer pursued, I can probably help implement it.

amalgawa avatar Apr 15 '24 09:04 amalgawa

@carlosalberto what is the status of this one?

svrnm avatar May 06 '24 10:05 svrnm

+1 need to see this

esemeniuc avatar Jun 14 '24 20:06 esemeniuc

Discussed in the 7/17/24 TC meeting. We're closing this for the time being for:

  • niche use case / low demand
  • lack of performance benchmarks indicating benefit
  • protocols really need binary support baked in
  • seems more appropriate to drive at W3C level, and that effort has delayed and stalled

If the W3C binary traceparent effort picks back up, we should re-open this issue and implement in OpenTelemetry.

jack-berg avatar Jul 17 '24 15:07 jack-berg