Skip to content

[9.0] Change GeoIpCache and EnrichCache to LongAdder (#132922)#133251

Merged
elasticsearchmachine merged 1 commit intoelastic:9.0from
szybia:backport/9.0/pr-132922
Aug 20, 2025
Merged

[9.0] Change GeoIpCache and EnrichCache to LongAdder (#132922)#133251
elasticsearchmachine merged 1 commit intoelastic:9.0from
szybia:backport/9.0/pr-132922

Conversation

@szybia
Copy link
Contributor

@szybia szybia commented Aug 20, 2025

Backports the following commits to 9.0:

From the javadocs:

> [`LongAdder`] is usually preferable to `AtomicLong` when multiple threads update a common sum that is used for purposes such as collecting statistics, not for fine-grained synchronization control.

Since we largely write stats and read very infrequently, let's avoid any lock-contention possibilities by switching to the more appropriate data type.
@szybia szybia added :Distributed/Ingest Node Execution or management of Ingest Pipelines >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Data Management (obsolete) DO NOT USE. This team no longer exists. labels Aug 20, 2025
@elasticsearchmachine elasticsearchmachine merged commit 9cec1b3 into elastic:9.0 Aug 20, 2025
19 checks passed
@szybia szybia deleted the backport/9.0/pr-132922 branch August 20, 2025 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug :Distributed/Ingest Node Execution or management of Ingest Pipelines Team:Data Management (obsolete) DO NOT USE. This team no longer exists. v9.0.7

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants