Skip to content

Conversation

@xurui-c
Copy link
Member

@xurui-c xurui-c commented Jul 26, 2024

No description provided.

@xurui-c xurui-c requested a review from a team as a code owner July 26, 2024 20:20
@xurui-c xurui-c requested a review from a team July 26, 2024 20:20
@sentry
Copy link

sentry bot commented Jul 26, 2024

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: src/sentry/utils/snuba.py

Function Unhandled Issue
_bulk_snuba_query RateLimitExceeded: Query on could not be run due to allocation policies, info: {'details': {'ConcurrentRateLimitAllo... ...
Event Count: 195
_bulk_snuba_query RateLimitExceeded: Query on could not be run due to allocation policies, info: {'details': {'ConcurrentRateLimitAllo... ...
Event Count: 187
_bulk_snuba_query RateLimitExceeded: Query on could not be run due to allocation policies, info: {'details': {'ReferrerGuardRailPolicy... ...
Event Count: 108
_bulk_snuba_query QueryExecutionError: DB::Exception: Attempt to read after eof: while receiving packet from snuba-errors-tiger-mz-2-1:9... ...
Event Count: 2
_bulk_snuba_query RateLimitExceeded: Query on could not be run due to allocation policies, info: {'details': {'ConcurrentRateLimitAllo... ...
Event Count: 1

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

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 26, 2024
@xurui-c xurui-c requested a review from armenzg July 26, 2024 21:10
@codecov
Copy link

codecov bot commented Jul 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.12%. Comparing base (fd53a31) to head (0e5bf4d).
Report is 11 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #75101      +/-   ##
==========================================
- Coverage   78.13%   78.12%   -0.01%     
==========================================
  Files        6751     6751              
  Lines      301303   301303              
  Branches    51843    51843              
==========================================
- Hits       235412   235397      -15     
- Misses      59565    59574       +9     
- Partials     6326     6332       +6     
Files Coverage Δ
src/sentry/utils/snuba.py 90.00% <100.00%> (ø)

... and 8 files with indirect coverage changes

@xurui-c xurui-c merged commit f9a20be into master Jul 29, 2024
@xurui-c xurui-c deleted the rachel/warning branch July 29, 2024 16:10
@github-actions github-actions bot locked and limited conversation to collaborators Aug 14, 2024
@xurui-c xurui-c changed the title Make it more obvious that throttled queries are just warnings, not errors [CapMan visibility] Make it more obvious that throttled queries are just warnings, not errors Nov 6, 2024
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.

4 participants