Skip to content

Conversation

phacops
Copy link
Contributor

@phacops phacops commented Aug 13, 2025

These fields were added to Relay already.

getsentry/relay#5059

@phacops phacops requested review from a team as code owners August 13, 2025 18:10
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 13, 2025
Copy link

codecov bot commented Aug 13, 2025

Codecov Report

❌ Patch coverage is 90.00000% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/testutils/cases.py 83.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #97790      +/-   ##
==========================================
+ Coverage   76.73%   80.64%   +3.91%     
==========================================
  Files        8589     8588       -1     
  Lines      378457   378402      -55     
  Branches    24644    24630      -14     
==========================================
+ Hits       290401   305159   +14758     
+ Misses      87684    72871   -14813     
  Partials      372      372              

@jan-auer jan-auer changed the title feat(span-buffer): Add downsampled_retention_days and sample rates to items feat(spans): Add downsampled_retention_days and sample rates to items Aug 13, 2025
cursor[bot]

This comment was marked as outdated.

@phacops phacops merged commit 6c19c53 into master Aug 13, 2025
65 checks passed
@phacops phacops deleted the pierre/spans-add-downsampled-retention-and-sample-rates branch August 13, 2025 23:31
priscilawebdev pushed a commit that referenced this pull request Aug 25, 2025
andrewshie-sentry pushed a commit that referenced this pull request Aug 26, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Aug 29, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants