Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.x] Estimate segment field usages (#112760) #112777

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

dnhatn
Copy link
Member

@dnhatn dnhatn commented Sep 11, 2024

Backports the following commits to 8.x:

We have introduced a new memory estimation method in serverless, based 
on the number of segments and the fields within them. This new approach
works well overall, but it still falls short in cases where most fields
are used more than once - for example, in both doc_values and postings,
or doc_values and points. This change exposes the total usage of fields
in segments, allowing us to adjust the memory estimate for these cases.
@dnhatn dnhatn added :StorageEngine/Mapping The storage related side of mappings >non-issue auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:StorageEngine labels Sep 11, 2024
@elasticsearchmachine elasticsearchmachine merged commit f217322 into elastic:8.x Sep 12, 2024
15 checks passed
@dnhatn dnhatn deleted the backport/8.x/pr-112760 branch September 12, 2024 00:17
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 >non-issue :StorageEngine/Mapping The storage related side of mappings Team:StorageEngine v8.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants