Skip to content

Conversation

@idegtiarenko
Copy link
Contributor

This adds test cases for matching closed indices and hidden with a list of expressions and with pattern.

This adds test cases for matching closed indices and hidden with a list of expressions and with pattern.
@idegtiarenko idegtiarenko requested a review from nik9000 July 4, 2025 07:32
@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 Jul 4, 2025
@elasticsearchmachine
Copy link
Collaborator

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

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

.prepareCreate(".hidden-index-1")
.setSettings(Settings.builder().put(IndexMetadata.SETTING_INDEX_HIDDEN, true))
);
indexRandom(true, ".hidden-index-1", 15);
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be good to have a test with .hidden-* as well. I guess it's supposed to return 15 as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

According to FROM *-index-1 it looks like we only matching non-hidden indices.
But with .hidden-* we supply a hidden pattern. Let me add it 👍

@idegtiarenko idegtiarenko merged commit dc46b79 into elastic:main Jul 7, 2025
34 checks passed
@idegtiarenko idegtiarenko deleted the additional_index_resolution_cases branch July 7, 2025 11:06
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