Skip to content

Conversation

@mrstork
Copy link
Contributor

@mrstork mrstork commented Nov 13, 2025

Summary

  • Replaces the JsonTraceSerializer with a local implementation to suit our needs more directly
  • Saves ~15ms in functions along tracing and non-tracing paths
  • node_module size: ~26mb -> ~12mb

Before:

6.28mb @opentelemetry/otlp-transformer
5.96mb @opentelemetry/semantic-conventions
2.82mb protobufjs
2.4mb  @types/node
1.89mb @opentelemetry/sdk-metrics
1.19mb @opentelemetry/api
0.88mb @opentelemetry/core
0.77mb @opentelemetry/sdk-trace-base
0.63mb @opentelemetry/instrumentation
0.55mb @opentelemetry/resources
0.53mb acorn
0.41mb @opentelemetry/sdk-logs
0.4mb  @opentelemetry/api-logs
0.17mb require-in-the-middle
0.15mb resolve
0.14mb @opentelemetry/propagator-b3
0.13mb cjs-module-lexer
0.13mb long
0.12mb import-in-the-middle
0.1mb  semver

After:

5.96mb @opentelemetry/semantic-conventions
1.19mb @opentelemetry/api
0.88mb @opentelemetry/core
0.77mb @opentelemetry/sdk-trace-base
0.63mb @opentelemetry/instrumentation
0.55mb @opentelemetry/resources
0.53mb acorn
0.2mb  @opentelemetry/api-logs
0.17mb require-in-the-middle
0.15mb resolve
0.14mb @opentelemetry/propagator-b3
0.13mb cjs-module-lexer
0.12mb import-in-the-middle
0.1mb  semver
0.08mb @opentelemetry/propagator-jaeger
0.06mb @opentelemetry/context-async-hooks
0.04mb debug
0.03mb @opentelemetry/sdk-trace-node
0.03mb acorn-import-attributes
0.03mb function-bind

@mrstork mrstork force-pushed the replace-exporter branch 5 times, most recently from f5c74a5 to 407fca2 Compare November 13, 2025 21:35
@mrstork mrstork marked this pull request as ready for review November 14, 2025 17:05
@mrstork mrstork requested a review from a team as a code owner November 14, 2025 17:05
@mrstork mrstork merged commit 2d6b30a into main Nov 17, 2025
17 checks passed
@mrstork mrstork deleted the replace-exporter branch November 17, 2025 16:47
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