Skip to content
This repository has been archived by the owner on Jul 31, 2023. It is now read-only.

Make it possible to replace the TraceID/SpanID generation algorithms #643

Closed
semistrict opened this issue Mar 27, 2018 · 3 comments
Closed

Comments

@semistrict
Copy link
Contributor

To support special formats as required by some backends (XRay) we should make the TraceID/SpanID algorithms pluggable.

See: census-instrumentation/opencensus-specs#64

@odeke-em
Copy link
Member

Am not sure what @bogdandrutu will think of this change but the last time I spoke with him, there was a reservation towards allowing this.

@semistrict
Copy link
Contributor Author

I think his concern was with replacing the algorithm entirely (which was my original suggestion). This is a more gradual change that users can opt in to.

@rakyll
Copy link
Contributor

rakyll commented Mar 28, 2018

We briefly chatted about this issue to depend on NewSpanWithRemoteParent with custom ID generation. Btw, Bogdan's concern was not to introduce a new generator.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

No branches or pull requests

3 participants