getsentry / sentry

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

perf: move interfaces and integrations to orjson #71055

Closed anonrig closed 3 weeks ago

anonrig commented 3 weeks ago

Part of orjson move to get rid of sentry.utils.json

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 82.92683% with 14 lines in your changes are missing coverage. Please review.

Project coverage is 77.88%. Comparing base (dfc0962) to head (3e7377d).

:exclamation: Current head 3e7377d differs from pull request most recent head 6bf3625

Please upload reports for the commit 6bf3625 to get more accurate results.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #71055 +/- ## =========================================== + Coverage 41.97% 77.88% +35.90% =========================================== Files 6519 6528 +9 Lines 290525 290851 +326 Branches 50278 50339 +61 =========================================== + Hits 121959 226519 +104560 + Misses 161830 58085 -103745 + Partials 6736 6247 -489 ``` | [Files](https://app.codecov.io/gh/getsentry/sentry/pull/71055?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry) | Coverage Δ | | |---|---|---| | [src/sentry/interfaces/security.py](https://app.codecov.io/gh/getsentry/sentry/pull/71055?src=pr&el=tree&filepath=src%2Fsentry%2Finterfaces%2Fsecurity.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry#diff-c3JjL3NlbnRyeS9pbnRlcmZhY2VzL3NlY3VyaXR5LnB5) | `89.36% <100.00%> (+19.14%)` | :arrow_up: | | [src/sentry/shared\_integrations/client/base.py](https://app.codecov.io/gh/getsentry/sentry/pull/71055?src=pr&el=tree&filepath=src%2Fsentry%2Fshared_integrations%2Fclient%2Fbase.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry#diff-c3JjL3NlbnRyeS9zaGFyZWRfaW50ZWdyYXRpb25zL2NsaWVudC9iYXNlLnB5) | `87.45% <100.00%> (+44.93%)` | :arrow_up: | | [.../sentry/shared\_integrations/exceptions/\_\_init\_\_.py](https://app.codecov.io/gh/getsentry/sentry/pull/71055?src=pr&el=tree&filepath=src%2Fsentry%2Fshared_integrations%2Fexceptions%2F__init__.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry#diff-c3JjL3NlbnRyeS9zaGFyZWRfaW50ZWdyYXRpb25zL2V4Y2VwdGlvbnMvX19pbml0X18ucHk=) | `91.89% <100.00%> (+27.92%)` | :arrow_up: | | [src/sentry/shared\_integrations/response/base.py](https://app.codecov.io/gh/getsentry/sentry/pull/71055?src=pr&el=tree&filepath=src%2Fsentry%2Fshared_integrations%2Fresponse%2Fbase.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry#diff-c3JjL3NlbnRyeS9zaGFyZWRfaW50ZWdyYXRpb25zL3Jlc3BvbnNlL2Jhc2UucHk=) | `71.18% <100.00%> (+28.81%)` | :arrow_up: | | [src/sentry/testutils/helpers/backups.py](https://app.codecov.io/gh/getsentry/sentry/pull/71055?src=pr&el=tree&filepath=src%2Fsentry%2Ftestutils%2Fhelpers%2Fbackups.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry#diff-c3JjL3NlbnRyeS90ZXN0dXRpbHMvaGVscGVycy9iYWNrdXBzLnB5) | `99.41% <100.00%> (+2.63%)` | :arrow_up: | | [...y/testutils/performance\_issues/event\_generators.py](https://app.codecov.io/gh/getsentry/sentry/pull/71055?src=pr&el=tree&filepath=src%2Fsentry%2Ftestutils%2Fperformance_issues%2Fevent_generators.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry#diff-c3JjL3NlbnRyeS90ZXN0dXRpbHMvcGVyZm9ybWFuY2VfaXNzdWVzL2V2ZW50X2dlbmVyYXRvcnMucHk=) | `94.59% <100.00%> (+24.32%)` | :arrow_up: | | [src/sentry/utils/assets.py](https://app.codecov.io/gh/getsentry/sentry/pull/71055?src=pr&el=tree&filepath=src%2Fsentry%2Futils%2Fassets.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry#diff-c3JjL3NlbnRyeS91dGlscy9hc3NldHMucHk=) | `100.00% <100.00%> (+20.83%)` | :arrow_up: | | [src/sentry/utils/codecs.py](https://app.codecov.io/gh/getsentry/sentry/pull/71055?src=pr&el=tree&filepath=src%2Fsentry%2Futils%2Fcodecs.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry#diff-c3JjL3NlbnRyeS91dGlscy9jb2RlY3MucHk=) | `100.00% <100.00%> (+28.88%)` | :arrow_up: | | [src/sentry/utils/email/message\_builder.py](https://app.codecov.io/gh/getsentry/sentry/pull/71055?src=pr&el=tree&filepath=src%2Fsentry%2Futils%2Femail%2Fmessage_builder.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry#diff-c3JjL3NlbnRyeS91dGlscy9lbWFpbC9tZXNzYWdlX2J1aWxkZXIucHk=) | `91.89% <100.00%> (+48.35%)` | :arrow_up: | | [src/sentry/utils/outcomes.py](https://app.codecov.io/gh/getsentry/sentry/pull/71055?src=pr&el=tree&filepath=src%2Fsentry%2Futils%2Foutcomes.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry#diff-c3JjL3NlbnRyeS91dGlscy9vdXRjb21lcy5weQ==) | `98.11% <100.00%> (+9.65%)` | :arrow_up: | | ... and [8 more](https://app.codecov.io/gh/getsentry/sentry/pull/71055?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry) | | ... and [4333 files with indirect coverage changes](https://app.codecov.io/gh/getsentry/sentry/pull/71055/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry)
getsentry-bot commented 3 weeks ago

PR reverted: 191f282684c8ce4f10c82b3232041a1f7934de8c