Skip to content

ESQL: use field_caps native nested fields filtering#121645

Merged
astefan merged 1 commit intoelastic:8.17from
astefan:117054__fix_8.17_backport
Feb 4, 2025
Merged

ESQL: use field_caps native nested fields filtering#121645
astefan merged 1 commit intoelastic:8.17from
astefan:117054__fix_8.17_backport

Conversation

@astefan
Copy link
Copy Markdown
Contributor

@astefan astefan commented Feb 4, 2025

Fixes #117054 by partially reverting #111495 and using -nested as a filter for the _field_caps call.

Backports #117201.

…7201) (elastic#117375)

* ESQL: use field_caps native nested fields filtering (elastic#117201)

* Just filter the nested fields natively with field_caps support
@astefan astefan added >bug backport auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Analytics/ES|QL AKA ESQL v8.17.2 labels Feb 4, 2025
@astefan astefan enabled auto-merge (squash) February 4, 2025 12:08
@astefan astefan requested review from alex-spies and nik9000 February 4, 2025 12:08
Copy link
Copy Markdown
Contributor

@alex-spies alex-spies left a comment

Choose a reason for hiding this comment

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

It seems this is an identical backport of #117201 except for 1 line (commas in the EsqlCapabilities enum). It adds a capability, but no transport version bump. Therefore, I think this should be fine to backport.

We could be extra paranoid and slap test-full-bwc on this, but since the backport to 8.x didn't cause trouble, full bwc tests for 8.17 shouldn't cause trouble, either.

@astefan astefan merged commit 73381db into elastic:8.17 Feb 4, 2025
@astefan astefan deleted the 117054__fix_8.17_backport branch February 4, 2025 13:28
astefan added a commit to astefan/elasticsearch that referenced this pull request Feb 6, 2025
…7201) (elastic#117375) (elastic#121645)

* Just filter the nested fields natively with field_caps support

(cherry picked from commit 73381db)
astefan added a commit that referenced this pull request Feb 6, 2025
* [8.x] ESQL: use field_caps native nested fields filtering (#117201) (#117375) (#121645)

* Just filter the nested fields natively with field_caps support

(cherry picked from commit 73381db)

* Add import
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 >bug v8.17.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants