search

getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
39.23k stars 4.21k forks source link

fix(merged-issues): Update grouphash endpoint to use simple serializer #81017

Closed snigdhas closed 23 hours ago

snigdhas commented 2 days ago

Switching to a simple serializer for the grouphash endpoint. This endpoint is used to load the merged-issues page on an issue and doesn't need to gather all the extra info (stacktrace, packages, tags, etc) about all events in order to populate the page.

Fixes https://github.com/getsentry/sentry/issues/80961

github-actions[bot] commented 2 days ago

🚨 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 #discuss-dev-infra channel.

codecov[bot] commented 1 day ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

:white_check_mark: All tests successful. No failed tests found.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #81017 +/- ## ========================================== - Coverage 78.28% 75.07% -3.22% ========================================== Files 7215 7208 -7 Lines 319794 319485 -309 Branches 44034 43974 -60 ========================================== - Hits 250359 239855 -10504 - Misses 63038 73096 +10058 - Partials 6397 6534 +137 ```