Opentelemetry-java: Rename ProbabilitySampler to TraceIdRatioBasedSampler

Created on 21 Aug 2020  路  5Comments  路  Source: open-telemetry/opentelemetry-java

Also verify we're meeting the spec after this merge:
https://github.com/open-telemetry/opentelemetry-specification/pull/611

good first issue help wanted p2 required-for-ga
Source
picture jkwatson

Most helpful comment

i can pick this to work on.

picture akshay1312 on 23 Aug 2020
👍2

All 5 comments

We should rename this at opentelemetry-proto repo (https://github.com/open-telemetry/opentelemetry-proto/issues/212) at the same time.

dengliming picture dengliming on 22 Aug 2020

i can pick this to work on.

akshay1312 picture akshay1312 on 23 Aug 2020
👍2

Hey @akshay1312 still working on this?

carlosalberto picture carlosalberto on 14 Sep 2020

I am willing to take this If no one has started yet.

dengliming picture dengliming on 15 Sep 2020
👍1

I have submitted a related PR here

dengliming picture dengliming on 17 Sep 2020
Was this page helpful?
0 / 5 - 0 ratings

Related issues

pavolloffay picture pavolloffay  路  5Comments
chrvo picture chrvo  路  4Comments
jkwatson picture jkwatson  路  3Comments
malafeev picture malafeev  路  3Comments
jkwatson picture jkwatson  路  4Comments