-
Notifications
You must be signed in to change notification settings - Fork 1
Investigate what sampler_type means and why this is unknown during check-traces tests #4
Comments
@pnaik1 I found these things in case you can find an answer faster than me: https://www.jaegertracing.io/docs/1.39/sampling/ https://opentelemetry.io/docs/reference/specification/trace/tracestate-probability-sampling/ |
jaegertracing/jaeger#2197 I think , this may be a help |
@pnaik1 that helps a lot thanks. After read that issue you shared, I could bet that the Let's keep this issue open then. |
Taking a look in 2023 👍
We don't need to worry about that for this repository. Either way I found this comment: open-telemetry/opentelemetry-js#693 (comment) UPDATE: This is not the correct reason to close this issue. But related to the link of the comment I shared, apparently the code has changed over time... |
This is the current result we have when running the CI tests on check-traces repository:
The text was updated successfully, but these errors were encountered: