Repository navigation
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 406c7ef. Configure here.
| == DownsampledStorageConfig.MODE_LOW_PRIORITY | ||
| ): | ||
| tier = max(tier, Tier.TIER_8) | ||
| return tier |
There was a problem hiding this comment.
Low priority flagged as routing mistake
Low Severity
MODE_LOW_PRIORITY now starts at TIER_8 by design, but _emit_routing_mistake still treats any non-TIER_1 query that finishes under the sampled-too-low threshold as a routing_mistake. Fast low-priority queries will emit false sampled_too_low metrics.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 406c7ef. Configure here.


Fixes EAP-865
MODE_LOW_PRIORITYuses the same OutcomesBased path asMODE_NORMAL, but starts at TIER_8 instead of TIER_1. Further downsample to TIER_64 / TIER_512 still happens when ingest is too large.https://linear.app/getsentry/issue/EAP-865/add-the-ability-to-query-low-fidelity-data