Skip to content

Add factories for SamplingResult#3410

Merged
jkwatson merged 7 commits intoopen-telemetry:mainfrom
Rocksnake:fix/add_factories_for_samplingResult
Jul 20, 2021
Merged

Add factories for SamplingResult#3410
jkwatson merged 7 commits intoopen-telemetry:mainfrom
Rocksnake:fix/add_factories_for_samplingResult

Conversation

@Rocksnake
Copy link
Copy Markdown
Contributor

@Rocksnake Rocksnake commented Jul 20, 2021

#3360

Fixes #3354

rebase it by re-create the branch and re-push

@Rocksnake Rocksnake requested a review from a user July 20, 2021 01:52
@anuraaga anuraaga changed the title fix:rebase it #3360 Add factories for SamplingResult Jul 20, 2021
Copy link
Copy Markdown
Contributor

@anuraaga anuraaga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :)

Comment thread sdk/trace/src/main/java/io/opentelemetry/sdk/trace/samplers/SamplingResult.java Outdated
Comment thread sdk/trace/src/main/java/io/opentelemetry/sdk/trace/samplers/SamplingResult.java Outdated
Comment thread sdk/trace/src/main/java/io/opentelemetry/sdk/trace/samplers/SamplingResult.java Outdated
Comment thread sdk/trace/src/main/java/io/opentelemetry/sdk/trace/samplers/SamplingResult.java Outdated
Comment thread sdk/trace/src/main/java/io/opentelemetry/sdk/trace/samplers/SamplingResult.java Outdated
Comment thread sdk/trace/src/main/java/io/opentelemetry/sdk/trace/samplers/SamplingResult.java Outdated
Comment thread sdk/trace/src/main/java/io/opentelemetry/sdk/trace/samplers/SamplingResult.java Outdated
@jkwatson jkwatson merged commit 9fdd001 into open-telemetry:main Jul 20, 2021
This was referenced Dec 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add factories for SamplingResult

3 participants