Skip to content

Conversation

henningandersen
Copy link
Contributor

Fix up the reads metric to also consider tryRead accessing a new region as a read/hit. Add cache miss metric that only increments once per region per read. Add a miss ratio as well for easy querying.

Fix up the reads metric to also consider tryRead accessing a new region
as a read/hit. Add cache miss metric that only increments once per region
per read. Add a miss ratio as well for easy querying.
@elasticsearchmachine elasticsearchmachine added the Team:Distributed Indexing Meta label for Distributed Indexing team label Aug 6, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-distributed-indexing (Team:Distributed Indexing)

@elasticsearchmachine
Copy link
Collaborator

Hi @henningandersen, I've created a changelog YAML for you.

@elasticsearchmachine elasticsearchmachine added the serverless-linked Added by automation, don't add manually label Aug 6, 2025
Copy link
Member

@tlrx tlrx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@andreidan andreidan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks Henning

@henningandersen henningandersen merged commit 444c5f2 into elastic:main Aug 7, 2025
33 checks passed
@henningandersen
Copy link
Contributor Author

Thanks both!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Indexing/Searchable Snapshots Searchable snapshots / frozen indices. >enhancement serverless-linked Added by automation, don't add manually Team:Distributed Indexing Meta label for Distributed Indexing team v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants