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 bug with significant terms background count #76730

Merged

Conversation

benwtrent
Copy link
Member

When building empty responses for shards that don't have the term field in question,
significant terms ignored the background filter.

This commit fixes this bug by respecting the background filter count, even
when building empty results.

closes #76729

@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Aug 19, 2021
@elasticmachine
Copy link
Collaborator

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

Copy link
Member

@nik9000 nik9000 left a comment

Choose a reason for hiding this comment

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

That looks right to me. If the new test is happy I'm happy.

Copy link
Contributor

@imotov imotov left a comment

Choose a reason for hiding this comment

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

LGTM2! Thanks a lot for addressing it so quickly and fully.

@imotov
Copy link
Contributor

imotov commented Aug 19, 2021

The test failure in elasticsearch-ci/part-2 is unrelated and is already tracked in #76689, restarting

@elasticmachine run elasticsearch-ci/part-2

@benwtrent benwtrent added the auto-backport Automatically create backport pull requests when merged label Aug 20, 2021
@benwtrent benwtrent merged commit 55edc62 into elastic:master Aug 20, 2021
@benwtrent benwtrent deleted the feature/aggs-sig-terms-bg-count-bug-fix branch August 20, 2021 11:21
benwtrent added a commit to benwtrent/elasticsearch that referenced this pull request Aug 20, 2021
When building empty responses for shards that don't have the term field in question,
significant terms ignored the background filter.

This commit fixes this bug by respecting the background filter count, even
when building empty results.

closes elastic#76729
benwtrent added a commit to benwtrent/elasticsearch that referenced this pull request Aug 20, 2021
When building empty responses for shards that don't have the term field in question,
significant terms ignored the background filter.

This commit fixes this bug by respecting the background filter count, even
when building empty results.

closes elastic#76729
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
7.x
7.15

benwtrent added a commit to benwtrent/elasticsearch that referenced this pull request Aug 20, 2021
benwtrent added a commit that referenced this pull request Aug 20, 2021
* Fix bug with significant terms background count (#76730)

When building empty responses for shards that don't have the term field in question,
significant terms ignored the background filter.

This commit fixes this bug by respecting the background filter count, even
when building empty results.

closes #76729
elasticsearchmachine pushed a commit that referenced this pull request Aug 20, 2021
* Fix bug with significant terms background count (#76730)

When building empty responses for shards that don't have the term field in question,
significant terms ignored the background filter.

This commit fixes this bug by respecting the background filter count, even
when building empty results.

closes #76729

* fixing bwc yaml test version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Aggregations Aggregations auto-backport Automatically create backport pull requests when merged >bug Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v7.15.1 v7.16.0 v8.0.0-alpha2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Significant terms ignores background filter
6 participants