Skip to content

[otlp-transformer] migrate protobuf trace serialization to use hand-rolled implementation #6569

@pichlermarc

Description

@pichlermarc

Description

We recently introduced a custom protobuf serializer for Logs, as well as a response deserializer. There were several reasons for this change:

  1. performance - a hand-rolled implementation avoids unnecessary transformations, and is therefore quicker and requires fewer allocations.
  2. bundle-size - since we need to do fewer transformations, and we don't need the auto-generated code anymore, our bundle size should shrink significantly
  3. avoiding protobuf.js production dependency - that dependency is the cause for use of eval is strongly discouraged #4987

This issue tracks implementing trace serialization based on the pattern and and serializer introduced by these two PRs

This issue is considered done when:

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions