Skip to content

Conversation

@nik9000
Copy link
Member

@nik9000 nik9000 commented May 19, 2025

When adding support for pushing == to semantic_text we were incorrectly asserting that all queries to that field used nested documents. That's normal for semantic_text, but sometimes we query indices that don't have any nested fields.

Closes #128122

When adding support for pushing `==` to `semantic_text` we were
incorrectly asserting that all queries to that field used nested
documents. That's normal for `semantic_text`, but sometimes we query
indices that don't have any nested fields.

Closes elastic#128122
@nik9000 nik9000 added >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Analytics/ES|QL AKA ESQL v9.1.0 labels May 19, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label May 19, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine elasticsearchmachine merged commit 12cb957 into elastic:main May 19, 2025
17 checks passed
@nik9000 nik9000 deleted the fix_128122 branch May 19, 2025 17:23
@nik9000
Copy link
Member Author

nik9000 commented May 19, 2025

Backported by #128156

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!) Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] PushQueriesIT testEqualityAndOther {semantic_text} failing

2 participants