Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Implement Sampling Seed Propagation #837

Open
kahest opened this issue Feb 4, 2025 · 1 comment
Open

Implement Sampling Seed Propagation #837

kahest opened this issue Feb 4, 2025 · 1 comment

Comments

@kahest
Copy link
Member

kahest commented Feb 4, 2025

Description

See https://github.com/getsentry/projects/issues/316

To increase the chance of capturing complete traces when users return a new sample rate tracesSampler in backend services, we propagate the random value used by the SDK for computing the sampling decision instead of creating a new random value in every service. Therefore, across a trace every SDK uses the same random value.

Develop: Spec

Similar to getsentry/sentry-react-native#4425 we most likely need to only bump the JS SDK.

See also

Sample implemeantion in PHP getsentry/sentry-php#1793.

@lucas-zimerman
Copy link
Collaborator

We will wait for an official V9 release since it was only implemented on it, if there is some urgency on it, we should port the change to V8.

@lucas-zimerman lucas-zimerman moved this from Backlog to Todo in Mobile & Cross Platform SDK Feb 4, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
Status: Todo
Development

No branches or pull requests

2 participants