-
Notifications
You must be signed in to change notification settings - Fork 833
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[otlp-transformer] reduce public API surface #4583
Comments
This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days. |
Most usages for the types that will be removed are in tests, I'm working on changing the tests so that they are not used anymore. (the start of that is #5196) |
See also open-telemetry/opentelemetry-js-contrib#2565 for a follow-up in contrib to replace usage of types that will be removed by completing this issue. |
Description
The
@opentelemetry/otlp-transformer
package contains utilities to serialize signals data to and from OTLP (binary protobuf) or OTLP (json). As all OTLP exporters depend on this package, stabilization of@opentelemetry/otlp-transformer
is required for OTLP exporter stability.The API surface currently consists of 40+ types and functions. Reducing the API surface is required to enable optimizations in the serialization process in terms memory use, performance, and bundle size without being constrained by a large stable public API.
Goal of this issue is to:
Move the full serialization process to
@opentelemetry/otlp-transformer
, this way we will be able to reduce the API surface to:ISerializer
interfaceThe text was updated successfully, but these errors were encountered: