Skip to content

Conversation

AbhiPrasad
Copy link
Member

Logs are GA now, so we can remove this note from the createSentryWinstonTransport method.

@AbhiPrasad AbhiPrasad self-assigned this Sep 12, 2025
Copy link
Contributor

node-overhead report 🧳

Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.

Scenario Requests/s % of Baseline Prev. Requests/s Change %
GET Baseline 8,990 - 9,405 -4%
GET With Sentry 1,348 15% 1,244 +8%
GET With Sentry (error only) 6,122 68% 6,082 +1%
POST Baseline 1,219 - 1,194 +2%
POST With Sentry 541 44% 496 +9%
POST With Sentry (error only) 1,076 88% 1,061 +1%
MYSQL Baseline 3,355 - 3,356 -0%
MYSQL With Sentry 493 15% 410 +20%
MYSQL With Sentry (error only) 2,734 81% 2,703 +1%

View base workflow run

@AbhiPrasad AbhiPrasad enabled auto-merge (squash) September 12, 2025 14:59
@AbhiPrasad AbhiPrasad merged commit abb9337 into develop Sep 12, 2025
260 of 263 checks passed
@AbhiPrasad AbhiPrasad deleted the abhi-winston-remove-note branch September 12, 2025 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants