Skip to content

Conversation

not-napoleon
Copy link
Member

This removes support for running date aggregations over boolean fields. This has never been useful, was deprecated in 7.x, and is finally being disabled for 9.0.

@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Dec 11, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@elasticsearchmachine
Copy link
Collaborator

Hi @not-napoleon, I've created a changelog YAML for you. Note that since this PR is labelled >breaking, you need to update the changelog YAML to fill out the extended information sections.

Copy link
Contributor

@iverase iverase left a comment

Choose a reason for hiding this comment

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

LGTM

@not-napoleon not-napoleon requested a review from a team as a code owner December 17, 2024 14:44
@not-napoleon not-napoleon merged commit 5d9c3a2 into elastic:main Dec 17, 2024
16 checks passed
rjernst pushed a commit to rjernst/elasticsearch that referenced this pull request Dec 18, 2024
This removes support for running date aggregations over boolean fields. This has never been useful, was deprecated in 7.x, and is finally being disabled for 9.0.
@leemthompo
Copy link
Contributor

@not-napoleon is this PR relevant to the serverless changelog? [FYI this question is based on 9.0 breaking changes]

@leemthompo
Copy link
Contributor

is this PR relevant to the serverless changelog?

I'm 99.999% it's not, this was a scripted comment for the 45 PRs with breaking changelogs for 9.0 😄

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

Labels

:Analytics/Aggregations Aggregations >breaking Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.0.0

4 participants