Brett McBride

Results 292 comments of Brett McBride

I went over what the java SIG does for configuration, and found that they don't have a way to provide input via a well-defined structure (xml/json/yaml). AFAICT, having an SDK...

Closing, as I believe all of the issues originally identified have been resolved or are no longer relevant due to refactoring which has taken place over the past 12 months.

> that still seems like quite unintuitive behavior, if not like a bug I don't think there is anything we can do code-wise about accidentally starting duplicate root spans, because...

I ran some benchmarks against baggage parsing, it's very fast (~2us per execution when run 10k times) and I couldn't see much room for improvement. It looks like the original...

Closing due to inactivity, please re-open or create a new issue if this is still relevant.

We have a [lowish-priority issue](https://github.com/open-telemetry/opentelemetry-php/issues/904) over in opentelemetry-php to try building/testing against arm64, and having access to an arm-based self-hosted runner seems like it would help us out. At the...

I really like this. Could you add an example of how a component could be configured using environment variables? If we can do that, then it looks like this could...

I think we can merge this. Let's annotate it as `experimental`, and then I'll try to apply my config PR over the top. edit: the SPI package looks nicely self-contained,...

> Yes (but it should stay independent from the SDK); should it be added to this repository? I think it should be in this repository, it doesn't feel right for...

> Currently the packages:validate:installation check is failing @Nevay submitted a PR against your branch which makes this command happy (per your first suggestion)