Santosh Cheler

Results 57 comments of Santosh Cheler

> I do have a question about browser support - > > CompressionStream is supported on majority of browsers, but not all. On browsers where this is not supported, the...

@pichlermarc all aspects of this PR are resolved, can you take a look? @martinkuba @MSNev can you approve this PR when you get a chance?

@pichlermarc can you review this PR when you get a chance? There is a commit https://github.com/open-telemetry/opentelemetry-js/commit/8a2e225997f0c8cb850ee500bcde415ed0e2ea31 from you that needs your attention.

@martinkuba 1. I suggest skipping option 3 as it touches TracerProvider and will require more discussion / approvals. 2. Option 2 looks very similar to [MultiSpanProcessor](https://github.com/open-telemetry/opentelemetry-java/blob/main/sdk/trace/src/main/java/io/opentelemetry/sdk/trace/MultiSpanProcessor.java) in opentelemetry-java, so you...

Hi, wanted to give an update on this topic, since some of us from the client-side-telemetry SIG have asked a few TC members to help us on the topic further....

@pichlermarc @legendecas @dyladan could one of you help check what the unit tests are failing for? I only see "Error: Process completed with exit code 3." at the end of...

@open-telemetry/javascript-approvers can you please review this PR?

> I think it is fine to move the util function as a member method for the web, as the send function is accessing many members of the collector and...

@legendecas I made the suggested changes, please take a look.

@legendecas all the suggested changes are now complete, can you see if anything else is needed or approve? Thanks.