Skip to content
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

Pluck replay_id out of dynamic sampling context and add it to context if it exists #1958

Closed
JoshFerge opened this issue Mar 21, 2023 · 0 comments · Fixed by #1983
Closed
Assignees
Labels

Comments

@JoshFerge
Copy link
Member

As part of Connecting Backend Errors With Replays, https://github.com/getsentry/replay-backend/issues/251, we decided that we will add replay_id to the dynamic sampling context (getsentry/sentry-javascript#7555).

In Relay, we should pluck the replay_id out of the dynamic sampling context and add it to the context of the event. This will then be written as a column in the various clickhouse tables. I will plan on making a PR in one of our next sprints.

Original RFC getsentry/rfcs#60

@JoshFerge JoshFerge self-assigned this Mar 21, 2023
@HazAT HazAT changed the title Pluck replay-id out of dynamic sampling context and add it to context if it exists Pluck replay_id out of dynamic sampling context and add it to context if it exists Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants