Skip to content

Conversation

@javanna
Copy link
Member

@javanna javanna commented Mar 31, 2025

Backports the following commits to 8.17:

…Fields (elastic#125849) (elastic#125951)

We have some tolerance wound how many bytes we report for these completion fields. But the
values depend on the distribution of the random values that determine how many docs get
an option field. This commit makes the test more precise by computing the real ratio
between docs that have the optional field and the total number of docs, so that we
can base assertion on more realistic expectations.

Closes elastic#123269
@javanna javanna added :Search Foundations/Search Catch all for Search Foundations >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport labels Mar 31, 2025
@elasticsearchmachine elasticsearchmachine merged commit 2a1adad into elastic:8.17 Mar 31, 2025
15 checks passed
@javanna javanna deleted the backport/8.17/pr-125951 branch March 31, 2025 16:21
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 :Search Foundations/Search Catch all for Search Foundations >test Issues or PRs that are addressing/adding tests v8.17.5

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants