ref(ds): Remove reservoir sampling rule#115365
Conversation
github-actions
Bot
added
Scope: Frontend
|
🚨 Warning: This pull request contains Frontend and Backend changes! It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently. Have questions? Please ask in the |
| type: 'factor' | 'sampleRate' | 'minimumSampleRate'; | ||
| value: number; | ||
| limit?: number; | ||
| }; | ||
| type: 'trace' | 'transaction'; | ||
| type: 'trace' | 'transaction' | 'project'; |
There was a problem hiding this comment.
Drive by fix, to add new types which we added to Relay but never to the frontend here
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
📊 Type Coverage Diff✅ No new type safety issues introduced. Coverage: 93.49% |
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.
Reviewed by Cursor Bugbot for commit d47adcb. Configure here.
3 participants
This is no longer exposed in the product. This is also the first step of removing the reservoir sampling code from Relay.