Brett McBride

Results 292 comments of Brett McBride

I think this is useful even without a pure php library. Whilst developing to add elastic APM to an existing codebase, I found it immensely helpful to install the PHP...

+1 we would like to see this feature implemented

FYI this is super-low priority for us, I changed our docker image to not use zts to work around this issue.

Also worth noting: forcing the use of `CurlHandler` breaks guzzle's async feature, all requests happen sequentially.

We use `environment` to label traces as production/test/dev etc, which I believe is its intended purpose (and we do have multiple environments' traces going into the one elastic instance, so...

I agree with `SimpleSpanProcessor::forceFlush()`, but the conditional flush in `BatchSpanProcessor::onEnd()` seems critical to the batching functionality. Is the issue that we're using the `forceFlush()` mechanism not for its intended purpose?

> Hi, Can anyone help me resolve this code coverage error? It's complaining about a decrease in code coverage, specifically in `SpanExporterTrait` - you can `make test-coverage` locally, you will...

I think this needs an example of how to use and configure a retry policy, so that users can see it in action and understand how to configure (and disable/turn...

> failed because of doLog() function in LogsMessagetrait.php I understand now... By adding `LogsMessagesTrait` to `SpanExporterTrait`, our `LoggerExporter` ends up with two methods named `doLog`. In that case, it does...