search

census-instrumentation / opencensus-csharp

Distributed tracing and stats collecting framework
https://opencensus.io
Apache License 2.0
139 stars 32 forks source link

ISampler.ShouldSample must take tracestate as an argument #99

Closed SergeyKanzhelev closed 5 years ago

SergeyKanzhelev commented 5 years ago

tracestate may carry sampling-related information. So ShouldSample method of ISampler must take it as an argument

SergeyKanzhelev commented 5 years ago

Actually tracestate of parent is already part of parentContext object. There is still a need to allow to change a tracestate in a sampler. But it is a different task. Closing for now