Carson Ip

Results 161 comments of Carson Ip

Thanks for the detailed feature request. I am under the impression that we're already doing TBS based on transaction duration (see [code](https://github.com/elastic/apm-server/blob/b0e8925894e4d2aecb11c01de0c5fe6058866502/x-pack/apm-server/sampling/groups.go#L167)), it is implicit and should work out of...

The attached documentation now redirects to observability, which is way better for maintenance. In docs v3, we should hide all the 7.x vs 8.x complexity, and make it 9.0+ configs...

To reproduce, run `go test ./... -run=^TestServerElasticsearchOutput$ -count 1000 -failfast`

@lucabelluccini thanks, that pretty much sums it up. > Introduction of TTL of TBS storage > Introduce it at APM Server standalone >https://github.com/elastic/apm-server/pull/15106 ✅ (8.16.3 / 8.17.1 / 8.18.0 /...

> For sampling.tail.ttl, will it make sense to document it as Pebble (9.0+) will enforce it ( ❓ ) or do we want to keep this setting hidden? In 9.0...

Closing this meta issue as all attached issues are closed.

The reason behind the bug, e.g. upgrading from 8.13.2 to 8.15.0 - apm-server always (unless `apm-server.data_streams.namespace` is set in apm-server.yml) issues the bulk request to `logs-apm.error-default`. It will be subject...

I'm fairly convinced that it is due to slow httpServer and grpcServer shutdown, as I managed to reproduce something similar with an artificial delay: ``` diff --git a/internal/beatcmd/beat.go b/internal/beatcmd/beat.go index...

To fix this bug, it requires a few changes 1. httpServer and grpcServer need to respect shutdown timeout. It cannot just wait indefinitely for connections to terminate. 2. But even...

https://github.com/elastic/apm-server/pull/14339 is merged, but keeping this issue open, as we want to double check if all the processors are fine with concurrent runs.