Skip to content

[8.x] Backport "ESQL: Add nulls support to Categorize (#117655)"#117716

Merged
elasticsearchmachine merged 1 commit intoelastic:8.xfrom
ivancea:backport-117655-categorize-nulls
Nov 28, 2024
Merged

[8.x] Backport "ESQL: Add nulls support to Categorize (#117655)"#117716
elasticsearchmachine merged 1 commit intoelastic:8.xfrom
ivancea:backport-117655-categorize-nulls

Conversation

@ivancea
Copy link
Contributor

@ivancea ivancea commented Nov 28, 2024

Manual 8.x backport of #117655

Handle nulls and empty strings (Which resolve to null) on Categorize grouping function.

Also, implement `seenGroupIds()`, which would fail some queries with nulls otherwise.
@ivancea ivancea added >non-issue backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Analytics/ES|QL AKA ESQL labels Nov 28, 2024
@elasticsearchmachine elasticsearchmachine merged commit 74cf2c6 into elastic:8.x Nov 28, 2024
@ivancea ivancea deleted the backport-117655-categorize-nulls branch November 28, 2024 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.18.0

2 participants