Skip to content

Conversation

@idegtiarenko
Copy link
Contributor

This adds more cases for index resolution:

  • exclusion patterns
  • filters (including ones causing empty result set)

@idegtiarenko idegtiarenko added >test Issues or PRs that are addressing/adding tests Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL v9.2.0 labels Oct 1, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@idegtiarenko idegtiarenko changed the title Test more resolution cases Test more index resolution cases Oct 1, 2025
containsString("Unknown index [no-such-index]"),
() -> run(syncEsqlQueryRequest().query("FROM no-such-index"))
);
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is existing test case, it was moved down

Copy link
Contributor

@luigidellaquila luigidellaquila left a comment

Choose a reason for hiding this comment

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

LGTM

@idegtiarenko idegtiarenko merged commit faf18ef into elastic:main Oct 1, 2025
34 checks passed
@idegtiarenko idegtiarenko deleted the more_index_resolution_test_cases branch October 1, 2025 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants