sentry icon indicating copy to clipboard operation
sentry copied to clipboard

fix(processing): Separate second query of percentage comparison

Open schew2381 opened this issue 1 year ago • 1 comments

Fixes ALRT-101

schew2381 avatar Jun 21 '24 01:06 schew2381

Codecov Report

Attention: Patch coverage is 79.41176% with 7 lines in your changes missing coverage. Please review.

Project coverage is 78.02%. Comparing base (46e5215) to head (5b15021). Report is 14 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #73099       +/-   ##
===========================================
- Coverage   88.20%   78.02%   -10.18%     
===========================================
  Files        2923     6638     +3715     
  Lines      181871   296913   +115042     
  Branches    32467    51132    +18665     
===========================================
+ Hits       160424   231678    +71254     
- Misses      15504    58961    +43457     
- Partials     5943     6274      +331     
Files Coverage Δ
src/sentry/rules/conditions/event_frequency.py 89.32% <100.00%> (-0.33%) :arrow_down:
src/sentry/testutils/fixtures.py 93.29% <100.00%> (+0.01%) :arrow_up:
src/sentry/rules/processing/delayed_processing.py 87.93% <78.12%> (+1.83%) :arrow_up:

... and 3717 files with indirect coverage changes

codecov[bot] avatar Jun 21 '24 02:06 codecov[bot]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ KeyError: '1d' sentry.rules.processing.delayed_processing View Issue

Did you find this useful? React with a 👍 or 👎

sentry[bot] avatar Jul 02 '24 17:07 sentry[bot]