-
Notifications
You must be signed in to change notification settings - Fork 854
Open
Labels
pkg:OpenTelemetry.Exporter.OpenTelemetryProtocolIssues related to OpenTelemetry.Exporter.OpenTelemetryProtocol NuGet packageIssues related to OpenTelemetry.Exporter.OpenTelemetryProtocol NuGet package
Description
Hi @paulomorgado ,
bufferis used to store the serialized protobuf message. Currently all existing APIs such asWriteStringWithTag,WriteDoubleWithTagare also usingbyte[]. If we want to change this, I think we can create a dedicated PR later? @rajkumar-rangaraj What do you think?
Isn't this method being added now? Why add to the "problem" instead of adding to the "solution"?
Have a look at my branch.
Originally posted by @paulomorgado in #6534 (comment)
For more details, see discussion in the linked PR.
paulomorgado, JunweiSUN and splitt3r
Metadata
Metadata
Assignees
Labels
pkg:OpenTelemetry.Exporter.OpenTelemetryProtocolIssues related to OpenTelemetry.Exporter.OpenTelemetryProtocol NuGet packageIssues related to OpenTelemetry.Exporter.OpenTelemetryProtocol NuGet package