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

Fix RandomSamplerIT#testRandomSamplerHistogram test input #88191

Conversation

benwtrent
Copy link
Member

Periodically, a single 10 value is included which causes a 3rd histogram bucket to be created with very few values. Meaning sampling wouldn't get any docs that hit that bucket (especially with the randomly included deleted docs).

The failures that I saw had the 10.0 bucket with just one document.

closes: #88108

@benwtrent benwtrent added >test Issues or PRs that are addressing/adding tests :Analytics/Aggregations Aggregations auto-backport-and-merge Automatically create backport pull requests and merge when ready v8.4.0 v8.3.2 labels Jun 29, 2022
@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Jun 29, 2022
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

@benwtrent benwtrent merged commit 56befd0 into elastic:master Jul 5, 2022
@benwtrent benwtrent deleted the test/fix-RandomSamplerIT.testRandomSamplerHistogram branch July 5, 2022 18:15
benwtrent added a commit to benwtrent/elasticsearch that referenced this pull request Jul 5, 2022
)

Periodically, a single 10 value is included which causes a 3rd histogram bucket to be created with very few values. Meaning sampling wouldn't get any docs that hit that bucket (especially with the randomly included deleted docs).

The failures that I saw had the 10.0 bucket with just one document.

closes: elastic#88108
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.3

elasticsearchmachine pushed a commit that referenced this pull request Jul 5, 2022
…88288)

Periodically, a single 10 value is included which causes a 3rd histogram bucket to be created with very few values. Meaning sampling wouldn't get any docs that hit that bucket (especially with the randomly included deleted docs).

The failures that I saw had the 10.0 bucket with just one document.

closes: #88108
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Aggregations Aggregations auto-backport-and-merge Automatically create backport pull requests and merge when ready Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.3.2 v8.4.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reproducible Failure in RandomSamplerIT.testRandomSamplerHistogram
4 participants