Carson Ip

Results 154 comments of Carson Ip

> If someone is interested in getting this to work (on X11 systems), I've hacked together something similar for next/previous: [neopostmodern@3424b29](https://github.com/neopostmodern/Penguin-Subtitle-Player/commit/3424b29a6853eb0ef86a79b3d9decfc9755058ad) I don't think it's suitable for a PR though,...

> Yeah, I guess it would become a pain. Because even if you conditionally included this for Linux systems only it would break for the Wayland folks, no? Possibly, I...

@dmitryax Thanks for the batch sender! I've been experimenting with it lately. In my experiments, the batch sender is producing inconsistent batch sizes which could be lower than desired due...

@dmitryax ## batch size & concurrency > edit: I'm starting to think, will it be fine if the batch sender waits up to FlushTimeout even when concurrency limit is reached...

@dmitryax question: what's the reason behind the concurrency check in the first place? If it is just to avoid meaningless waits when all goroutines are blocked by the same batch,...

+1 on documentation. Here are the automation jobs that should be triggered manually: https://github.com/elastic/apm-server/actions/workflows/run-minor-release.yml https://github.com/elastic/apm-server/actions/workflows/run-patch-release.yml

Waiting for https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/32585 to merge, then it should come out of draft.

There was another bump in https://github.com/elastic/apm-server/pull/12179 backported to 8.12 - https://github.com/elastic/go-docappender/compare/c69a1cf8ce35...44217001fa53

```diff diff --git a/NOTICE.txt b/NOTICE.txt index 30c9e6c0c..51405e382 100644 --- a/NOTICE.txt +++ b/NOTICE.txt @@ -1,5 +1,5 @@ Elastic APM Server -Copyright 2014-2023 Elasticsearch BV +Copyright 2014-2024 Elasticsearch BV This product includes...