-
Notifications
You must be signed in to change notification settings - Fork 765
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
An example / proof of concept for stratified sampling in OpenTelemetry.NET #4208
An example / proof of concept for stratified sampling in OpenTelemetry.NET #4208
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #4208 +/- ##
==========================================
+ Coverage 83.27% 84.72% +1.44%
==========================================
Files 296 295 -1
Lines 11685 11475 -210
==========================================
- Hits 9731 9722 -9
+ Misses 1954 1753 -201
|
….com/kalyanaj/opentelemetry-dotnet into kalyanaj-stratified-sampling-example
docs/trace/advanced/stratified-sampling-example/stratified-sampling-example.csproj
Outdated
Show resolved
Hide resolved
…pling-example.csproj Co-authored-by: Timothy Mothra <tilee@microsoft.com>
docs/trace/advanced/stratified-sampling-example/stratified-sampling-example.csproj
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Couple of nits, not blockers
@utpilla one more person proof reading would be good before merge. |
This is an example / proof of concept for a basic stratified sampling in OpenTelemetry.NET.
Fixes #4207
Changes
Certain customers using OpenTelemetry want to do achieve stratified sampling. Stratified sampling is a way to divide a population (e.g., "all queries to a service") into mutually exclusive sub-populations aka "strata". For example, the strata here could be "all user-initiated queries", "all programmatic queries". Each stratum is then sampled using a probabilistic sampling method. This ensures that all sub-populations are represented.
For significant contributions please make sure you have completed the following items:
CHANGELOG.md
updated for non-trivial changes