Carson Ip

Results 98 comments of Carson Ip

> @carsonip is [this](https://github.com/elastic/apm-server/issues/11266#issuecomment-1651432073) the final version that has been implemented? @gbamparop The changes never got into 8.10 / main. They are still in a now-outdated draft PR: https://github.com/elastic/apm-aggregation/pull/43 The...

As seen in the [change here](https://github.com/elastic/apm-aggregation/pull/103/files#diff-ccd3edc253beb864df75e524a5f46b9580994c05e09b9e3d29d034eec8d960bcR1061) by @kruskall , `transaction.type: _other` is used to denote the overflow bucket of service instance transaction. @achyutjhunjhunwala I know we haven't completed the discussion...

Files need to be updated: - docs/access-api-keys.asciidoc - docs/feature-roles.asciidoc

I think in 40be17728b5be682559eb1f0bbde45f67ca4758f when we stopped using `MakeDefaultSupport`, we removed the publish event log line because `MakeDefaultSupport` sets up `debugPrintProcessor`: https://github.com/elastic/beats/blob/main/libbeat/publisher/processing/processors.go#L193

> Given that apmzap is meant for correlating logs with traces, shouldn't the tracer be initialised by instrumentation independently of logging? Yes that makes sense. @lahsivjar and I assumed that...

> drop the number of active requests by the number of released goroutines. That would be significantly more complicated, though. If you can try this out, that would be great...

> I assume the latest is supposed to test the latest 7.x and 8.x versions that are not yet released. We need to have smoke tests running for unreleased versions....

Is this issue solved in this commit https://github.com/glasslion/redlock/commit/56dc4316f83440cb2befa74b8dffc9d5635195a8 ? If so, should this issue be closed?

Expecting a lot of README merge conflicts with #33367