Skip to content

Conversation

@chargome
Copy link
Member

We were emitting the non-processed metric in the hook before – I changed this behaviour + added a test to verify.

@chargome chargome self-assigned this Nov 17, 2025
@github-actions
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.
⚠️ Warning: Base artifact is not the latest one, because the latest workflow run is not done yet. This may lead to incorrect results. Try to re-run all tests to get up to date results.

Scenario Requests/s % of Baseline Prev. Requests/s Change %
GET Baseline 8,619 - 8,592 +0%
GET With Sentry 1,325 15% 1,337 -1%
GET With Sentry (error only) 5,962 69% 6,047 -1%
POST Baseline 1,167 - 1,193 -2%
POST With Sentry 494 42% 507 -3%
POST With Sentry (error only) 1,043 89% 1,031 +1%
MYSQL Baseline 3,237 - 3,330 -3%
MYSQL With Sentry 445 14% 410 +9%
MYSQL With Sentry (error only) 2,654 82% 2,700 -2%

View base workflow run

@chargome chargome merged commit 8efeeeb into develop Nov 17, 2025
382 of 384 checks passed
@chargome chargome deleted the cg-emit-processed-metric branch November 17, 2025 09:57
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.

3 participants