Skip to content

Commit d2f4f47

Browse files
authored
feat(capman): Remove BytesScannedWindowAllocationPolicy from errors_ro (#6866)
It has been disabled the last three days and survived peak traffic just fine. For more info see the [policy consolidation](https://www.notion.so/sentry/CapMan-Visibility-Project-c16b375ed2ee43c19cd2fa163fc93332?pvs=4#185824a1f34b4ff6b31fe1c6ea872aca) document
1 parent 395db9c commit d2f4f47

File tree

2 files changed

+0
-20
lines changed

2 files changed

+0
-20
lines changed

snuba/datasets/configuration/events/storages/errors_ro.yaml

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -279,15 +279,6 @@ allocation_policies:
279279
getsentry.tasks.backfill_grouping_records:
280280
max_threads: 7
281281
concurrent_limit: 60
282-
- name: BytesScannedWindowAllocationPolicy
283-
args:
284-
required_tenant_types:
285-
- organization_id
286-
- referrer
287-
default_config_overrides:
288-
is_enforced: 1
289-
throttled_thread_number: 1
290-
org_limit_bytes_scanned: 10000000
291282
query_processors:
292283
- processor: UniqInSelectAndHavingProcessor
293284
- processor: TupleUnaliaser

tests/web/test_db_query.py

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -374,17 +374,6 @@ def test_db_query_success() -> None:
374374
"quota_unit": NO_UNITS,
375375
"suggestion": NO_SUGGESTION,
376376
},
377-
"BytesScannedWindowAllocationPolicy": {
378-
"can_run": True,
379-
"max_threads": 10,
380-
"explanation": {"storage_key": "StorageKey.ERRORS_RO"},
381-
"is_throttled": False,
382-
"throttle_threshold": 10000000,
383-
"rejection_threshold": MAX_THRESHOLD,
384-
"quota_used": 0,
385-
"quota_unit": "bytes",
386-
"suggestion": "The feature, organization/project is scanning too many bytes, this usually means they are abusing that API",
387-
},
388377
},
389378
}
390379

0 commit comments

Comments
 (0)