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

3 participants