Skip to content

Conversation

@chargome
Copy link
Member

Reverts #18759

we can merge in once open-telemetry/opentelemetry-js#6257 is out

@chargome chargome self-assigned this Jan 12, 2026
@linear
Copy link

linear bot commented Jan 12, 2026

Issue reopened: JS-1430 vercel-edge using node.js api

@chargome chargome requested a review from andreiborza January 12, 2026 09:32
@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.

Scenario Requests/s % of Baseline Prev. Requests/s Change %
GET Baseline 9,132 - 8,807 +4%
GET With Sentry 1,700 19% 1,708 -0%
GET With Sentry (error only) 6,076 67% 6,009 +1%
POST Baseline 1,186 - 1,181 +0%
POST With Sentry 594 50% 578 +3%
POST With Sentry (error only) 1,031 87% 1,039 -1%
MYSQL Baseline 3,338 - 3,295 +1%
MYSQL With Sentry 491 15% 450 +9%
MYSQL With Sentry (error only) 2,753 82% 2,669 +3%

View base workflow run

Copy link
Member

@andreiborza andreiborza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 🙏

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